[PATCH] D148841: [LV] Use SCEV for uniformity analysis across VF
Nikita Popov via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Apr 25 14:08:39 PDT 2023
nikic added inline comments.
================
Comment at: llvm/lib/Analysis/LoopAccessAnalysis.cpp:2570
+ // Build a new AddRec by multiplying the step by StepMultiplier and adding
+ // Offset * Step to the resulting AddRec.
+ auto *Ty = Expr->getType();
----------------
Assert that the addrec loop is correct?
================
Comment at: llvm/lib/Analysis/LoopAccessAnalysis.cpp:2577
+ SE.getMulExpr(StepC, Expr->getStepRecurrence(SE)),
+ Expr->getLoop(), Expr->getNoWrapFlags()),
+ SE.getMulExpr(OffsetC, Expr->getStepRecurrence(SE)));
----------------
Why is it valid to preserve nowrap flags here?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D148841/new/
https://reviews.llvm.org/D148841
More information about the llvm-commits
mailing list