This is an archive of the discontinued LLVM Phabricator instance.

[ARM] The ARM backend is MachineVerifier clean now.
ClosedPublic

Authored by fhahn on Aug 1 2017, 8:22 AM.

Reviewers

Commits

Summary

Thanks everyone involved in fixing the outstanding issues.

Yay, this is great!

lib/Target/ARM/ARMTargetMachine.h
66–68	You can just remove the whole override instead.

This revision is now accepted and ready to land.Aug 1 2017, 9:10 AM

Remove function

Path

Size

lib/

Target/

ARM/

2 lines

Show First 20 Lines • Show All 57 Lines • ▼ Show 20 Lines	public:

// Pass Pipeline Configuration		// Pass Pipeline Configuration
TargetPassConfig *createPassConfig(PassManagerBase &PM) override;		TargetPassConfig *createPassConfig(PassManagerBase &PM) override;

TargetLoweringObjectFile *getObjFileLowering() const override {		TargetLoweringObjectFile *getObjFileLowering() const override {
return TLOF.get();		return TLOF.get();
}		}

bool isMachineVerifierClean() const override {		bool isMachineVerifierClean() const override {
return false;		return true;
}		}
		MatzeBUnsubmitted Done Reply Inline Actions You can just remove the whole override instead. MatzeB: You can just remove the whole override instead.
};		};

/// ARM/Thumb little endian target machine.		/// ARM/Thumb little endian target machine.
///		///
class ARMLETargetMachine : public ARMBaseTargetMachine {		class ARMLETargetMachine : public ARMBaseTargetMachine {
public:		public:
ARMLETargetMachine(const Target &T, const Triple &TT, StringRef CPU,		ARMLETargetMachine(const Target &T, const Triple &TT, StringRef CPU,
StringRef FS, const TargetOptions &Options,		StringRef FS, const TargetOptions &Options,
Show All 17 Lines