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

[Nikita Popov via Phabricator via llvm-commits]

nikic requested changes to this revision.
nikic added inline comments.
This revision now requires changes to proceed.


================
Comment at: llvm/lib/Transforms/IPO/SCCP.cpp:215
 
-        MadeChanges = true;
-
-        if (&BB != &F.front())
+        if (&BB != &F.front()) {
+          MadeChanges = true;
----------------
Even in this case we're going to do a changeToUnreachable() below, so it's still changed?


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

https://reviews.llvm.org/D144838