[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