[PATCH] D104254: [VPlan] Support sinking recipes with uniform users outside sink target.

[Florian Hahn via Phabricator via llvm-commits]

fhahn added a comment.

In D104254#3000311 <https://reviews.llvm.org/D104254#3000311>, @Ayal wrote:

> Ship it!
> Does this turn the original sinkScalarOperands to be "all dead", and should therefore be retired?
> May be good to point out that this replacement does involve a (slight) functional change, due to the discrepancy in handling lane zero.

I think there's one more case we need to tackle before: IV's with vector & scalar users. Currently there will be a single 'widen-induction' recipe, which generates the vector & scalar values of the IV, so we cannot sink the scalar part in the VPlan transform yet.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D104254