[llvm] delete MF.verify from PPCMIPeephole pass (PR #108075)

via llvm-commits llvm-commits at lists.llvm.org
Tue Sep 10 12:49:08 PDT 2024


llvmbot wrote:


<!--LLVM PR SUMMARY COMMENT-->

@llvm/pr-subscribers-backend-powerpc

Author: zhijian lin (diggerlin)

<details>
<summary>Changes</summary>

delete  MF.verify from PPCMIPeephole pass , verifying the machine instructions of the functions in the PPCMIPeephole pass will cause a huge compile time increase in some cases.

---
Full diff: https://github.com/llvm/llvm-project/pull/108075.diff


1 Files Affected:

- (modified) llvm/lib/Target/PowerPC/PPCMIPeephole.cpp (-5) 


``````````diff
diff --git a/llvm/lib/Target/PowerPC/PPCMIPeephole.cpp b/llvm/lib/Target/PowerPC/PPCMIPeephole.cpp
index b8abee76cdfa80..b00984bac7ec75 100644
--- a/llvm/lib/Target/PowerPC/PPCMIPeephole.cpp
+++ b/llvm/lib/Target/PowerPC/PPCMIPeephole.cpp
@@ -177,11 +177,6 @@ struct PPCMIPeephole : public MachineFunctionPass {
     if (skipFunction(MF.getFunction()))
       return false;
     bool Changed = simplifyCode();
-#ifndef NDEBUG
-    if (Changed)
-      MF.verify(this, "Error in PowerPC MI Peephole optimization, compile with "
-                      "-mllvm -disable-ppc-peephole");
-#endif
     return Changed;
   }
 };

``````````

</details>


https://github.com/llvm/llvm-project/pull/108075


More information about the llvm-commits mailing list