[PATCH] D144838: [SCCP] Correct the made changes behavior

[chenglin.bi via Phabricator via llvm-commits]

bcl5980 updated this revision to Diff 500977.

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D144838/new/

https://reviews.llvm.org/D144838

Files:
  llvm/lib/Transforms/IPO/SCCP.cpp


Index: llvm/lib/Transforms/IPO/SCCP.cpp
===================================================================
--- llvm/lib/Transforms/IPO/SCCP.cpp
+++ llvm/lib/Transforms/IPO/SCCP.cpp
@@ -212,10 +212,10 @@
         LLVM_DEBUG(dbgs() << "  BasicBlock Dead:" << BB);
         ++NumDeadBlocks;
 
-        MadeChanges = true;
-
-        if (&BB != &F.front())
+        if (&BB != &F.front()) {
+          MadeChanges = true;
           BlocksToErase.push_back(&BB);
+        }
         continue;
       }
 


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D144838.500977.patch
Type: text/x-patch
Size: 501 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20230228/6c06930d/attachment.bin>