[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