[PATCH] D62625: LFTR for multiple exit loops

Philip Reames via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jun 13 11:38:09 PDT 2019


reames marked an inline comment as done.
reames added inline comments.


================
Comment at: lib/Transforms/Scalar/IndVarSimplify.cpp:2634
 
       const SCEV *BETakenCount = SE->getExitCount(L, ExitingBB);
       if (isa<SCEVCouldNotCompute>(BETakenCount))
----------------
nikic wrote:
> Rename BETakenCount to ExitCount here and also in linearFunctionTestReplace, possibly as NFC precommit? With multiple exits, this is not the backedge count anymore...
Done in 363293.


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

https://reviews.llvm.org/D62625





More information about the llvm-commits mailing list