[PATCH] D54021: [LoopSimplifyCFG] Teach LoopSimplifyCFG to constant-fold branches and switches

Anna Thomas via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Nov 16 09:27:02 PST 2018


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

LGTM. Thanks for working through the comments.



================
Comment at: lib/Transforms/Scalar/LoopSimplifyCFG.cpp:90
+  // current loop and its preheader and do not affect preheader's reachibility
+  // from any other block. So in practice this variable set to true means that
+  // loop's latch has become unreachable from loop header.
----------------
Minor: can remove the "in practice" here.


https://reviews.llvm.org/D54021





More information about the llvm-commits mailing list