[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