[PATCH] D66890: [IndVarSimplify] Do not use SCEV expander for IVCount in LFTR when possible.

Dávid Bolvanský via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Aug 28 09:34:55 PDT 2019


xbolva00 added subscribers: reames, xbolva00.
xbolva00 added a comment.

Can you run  ./update_test_checks.py  on llvm/test/Transforms/IndVarSimplify/add_nsw.ll and llvm/test/Transforms/IndVarSimplify/udiv.ll?

cc @reames since he is more familiar with SCEV and can comment this patch.



================
Comment at: llvm/lib/Transforms/Scalar/IndVarSimplify.cpp:2399
+    BranchInst *BI = cast<BranchInst>(ExitingBB->getTerminator());
+    if (ICmpInst *Cmp = dyn_cast<ICmpInst>(BI->getOperand(0))) {
+      Value *Limit = Cmp->getOperand(0);
----------------
nits:
auto *BI

auto *Cmp


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D66890





More information about the llvm-commits mailing list