This is an archive of the discontinued LLVM Phabricator instance.

[PowerPC] Remove the override of isMachineVerifierClean() to open machine verifier
ClosedPublic

Authored by • wuzish on Mar 5 2019, 10:07 PM.

Reviewers

Commits

Summary

After fix all asserts found by machine verifier in PowerPC target with following patches, we can activate machine verifier as default.

Thanks to everyone who worked on this! LGTM.

This revision is now accepted and ready to land.Mar 7 2019, 3:50 AM

This revision was automatically updated to reflect the committed changes.

Path

Size

llvm/

trunk/

lib/

Target/

PowerPC/

4 lines

Show First 20 Lines • Show All 52 Lines • ▼ Show 20 Lines	public:
TargetLoweringObjectFile *getObjFileLowering() const override {		TargetLoweringObjectFile *getObjFileLowering() const override {
return TLOF.get();		return TLOF.get();
}		}
bool isELFv2ABI() const { return TargetABI == PPC_ABI_ELFv2; }		bool isELFv2ABI() const { return TargetABI == PPC_ABI_ELFv2; }
bool isPPC64() const {		bool isPPC64() const {
const Triple &TT = getTargetTriple();		const Triple &TT = getTargetTriple();
return (TT.getArch() == Triple::ppc64 \|\| TT.getArch() == Triple::ppc64le);		return (TT.getArch() == Triple::ppc64 \|\| TT.getArch() == Triple::ppc64le);
};		};

bool isMachineVerifierClean() const override {
return false;
}
};		};
} // end namespace llvm		} // end namespace llvm

#endif		#endif