[PATCH] D87344: [IndVars] Remove exiting conditions that are trivially true/false

[Max Kazantsev via Phabricator via llvm-commits]

mkazantsev added inline comments.


================
Comment at: llvm/lib/Transforms/Scalar/IndVarSimplify.cpp:2368
 
   // Remove all exits which aren't both rewriteable and analyzeable.
   auto NewEnd = llvm::remove_if(ExitingBlocks, [&](BasicBlock *ExitingBB) {
----------------
nikic wrote:
> Nit: The mention of analyzeable exits here is stale now.
Well, they are kind of still analyzable. We just do not know exact exit count for them, but it does not mean we know absolutely nothing.


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

https://reviews.llvm.org/D87344