[PATCH] D36012: Update phi nodes in LowerTypeTests control flow simplification

Peter Collingbourne via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jul 28 13:06:31 PDT 2017


pcc accepted this revision.
pcc added a comment.
This revision is now accepted and ready to land.

LGTM



================
Comment at: lib/Transforms/IPO/LowerTypeTests.cpp:638
+        // Update phis in Else resulting from InitialBB being split
+        for (auto &phi : Else->phis())
+          phi.addIncoming(phi.getIncomingValueForBlock(Then), InitialBB);
----------------
Nit: rename to `Phi`.


https://reviews.llvm.org/D36012





More information about the llvm-commits mailing list