[PATCH] D123720: [VPlan] Replace use of needsVectorIV with VPlan user check.

[Florian Hahn via Phabricator via llvm-commits]

fhahn added a comment.

In D123720#3449843 <https://reviews.llvm.org/D123720#3449843>, @Ayal wrote:

> Looks good to me.
> Can this too lead to different - more accurate decisions? If so wonder if that could be tested(?)

The code was already falling back to `usesScalars` in the loop below, if `needsVectorIV` was returning true. In the committed version I completely dropped the check entirely. I wasn't able to find a case where this would lead to more accurate decisions unfortunately.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D123720