[PATCH] D62625: LFTR for multiple exit loops

Nikita Popov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jun 13 10:10:50 PDT 2019


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

LGTM, thanks for the extra tests.



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


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

https://reviews.llvm.org/D62625





More information about the llvm-commits mailing list