[PATCH] D59011: [PowerPC] Remove the override of isMachineVerifierClean() to open machine verifier

[Zixuan Wu via Phabricator via llvm-commits]

wuzish created this revision.
wuzish added reviewers: hfinkel, nemanjai, kbarton, jsji.
Herald added a project: LLVM.
Herald added a subscriber: llvm-commits.

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

It's also found in PR#27456, https://bugs.llvm.org/show_bug.cgi?id=27456


Repository:
  rL LLVM

https://reviews.llvm.org/D59011

Files:
  lib/Target/PowerPC/PPCTargetMachine.h


Index: lib/Target/PowerPC/PPCTargetMachine.h
===================================================================
--- lib/Target/PowerPC/PPCTargetMachine.h
+++ lib/Target/PowerPC/PPCTargetMachine.h
@@ -58,10 +58,6 @@
     const Triple &TT = getTargetTriple();
     return (TT.getArch() == Triple::ppc64 || TT.getArch() == Triple::ppc64le);
   };
-
-  bool isMachineVerifierClean() const override {
-    return false;
-  }
 };
 } // end namespace llvm
 


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D59011.189446.patch
Type: text/x-patch
Size: 452 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20190306/0d4a866a/attachment.bin>