[PATCH] D128755: [VPlan] Make sure optimizeInductions removes wide ind from scalar plan.
Bardia Mahjour via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Jun 28 15:37:00 PDT 2022
bmahjour added inline comments.
================
Comment at: llvm/lib/Transforms/Vectorize/VPlanTransforms.cpp:388
if (!IV ||
- all_of(IV->users(), [IV](VPUser *U) { return !U->usesScalars(IV); }))
+ (all_of(IV->users(), [IV](VPUser *U) { return !U->usesScalars(IV); }) &&
+ !NeedsScalarSteps))
----------------
suggestion:
```
(none_of(IV->users(), [IV](VPUser *U) { return U->usesScalars(IV); })
```
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D128755/new/
https://reviews.llvm.org/D128755
More information about the llvm-commits
mailing list