Refactor condition in SimplifyCFG.cpp

wsmoses · web-flow · commit a79af80d7aa8 · 2025-09-13T18:22:58.000-05:00
diff --git a/llvm/lib/Transforms/Utils/SimplifyCFG.cpp b/llvm/lib/Transforms/Utils/SimplifyCFG.cpp
@@ -3393,7 +3393,7 @@ bool SimplifyCFGOpt::speculativelyExecuteBB(BranchInst *BI,
   // Similarly strip attributes that maybe dependent on condition we are
   // hoisting above.
   for (auto &I : make_early_inc_range(*ThenBB)) {
-    if (!SpeculatedStoreValue || &I != SpeculatedStore && !isa<CallBase>(&I)) {
+    if ((!SpeculatedStoreValue || &I != SpeculatedStore) && !isa<CallBase>(&I)) {
       I.setDebugLoc(DebugLoc::getDropped());
     }
     I.dropUBImplyingAttrsAndMetadata();

Original file line number	Diff line number	Diff line change
`@@ -3393,7 +3393,7 @@ bool SimplifyCFGOpt::speculativelyExecuteBB(BranchInst *BI,`
`3393`	`3393`	`// Similarly strip attributes that maybe dependent on condition we are`
`3394`	`3394`	`// hoisting above.`
`3395`	`3395`	`for (auto &I : make_early_inc_range(*ThenBB)) {`
`3396`		`- if (!SpeculatedStoreValue \|\| &I != SpeculatedStore && !isa<CallBase>(&I)) {`
	`3396`	`+ if ((!SpeculatedStoreValue \|\| &I != SpeculatedStore) && !isa<CallBase>(&I)) {`
`3397`	`3397`	`I.setDebugLoc(DebugLoc::getDropped());`
`3398`	`3398`	`}`
`3399`	`3399`	`I.dropUBImplyingAttrsAndMetadata();`