[llvm] cfeb0bf - [SimpleLoopUnswitch] Temorarily switch off simple-loop-unswitch-inject-invariant-conditions. PR60736

Max Kazantsev via llvm-commits llvm-commits at lists.llvm.org
Tue Feb 14 03:23:30 PST 2023


Author: Max Kazantsev
Date: 2023-02-14T18:23:11+07:00
New Revision: cfeb0bf83ec160c2a7fc40313752c9a9407eaae2

URL: https://github.com/llvm/llvm-project/commit/cfeb0bf83ec160c2a7fc40313752c9a9407eaae2
DIFF: https://github.com/llvm/llvm-project/commit/cfeb0bf83ec160c2a7fc40313752c9a9407eaae2.diff

LOG: [SimpleLoopUnswitch] Temorarily switch off simple-loop-unswitch-inject-invariant-conditions. PR60736

It caused an assertion failure, not sure induced or introduced. Disabling
to investigate it. See details at https://github.com/llvm/llvm-project/issues/60736

Added: 
    

Modified: 
    llvm/lib/Transforms/Scalar/SimpleLoopUnswitch.cpp

Removed: 
    


################################################################################
diff  --git a/llvm/lib/Transforms/Scalar/SimpleLoopUnswitch.cpp b/llvm/lib/Transforms/Scalar/SimpleLoopUnswitch.cpp
index 2fda1ac0bee7..ee0cffe6271a 100644
--- a/llvm/lib/Transforms/Scalar/SimpleLoopUnswitch.cpp
+++ b/llvm/lib/Transforms/Scalar/SimpleLoopUnswitch.cpp
@@ -125,7 +125,7 @@ static cl::opt<bool> InjectInvariantConditions(
     "simple-loop-unswitch-inject-invariant-conditions", cl::Hidden,
     cl::desc("Whether we should inject new invariants and unswitch them to "
              "eliminate some existing (non-invariant) conditions."),
-    cl::init(true));
+    cl::init(false));
 
 static cl::opt<unsigned> InjectInvariantConditionHotnesThreshold(
     "simple-loop-unswitch-inject-invariant-condition-hotness-threshold",


        


More information about the llvm-commits mailing list