[PATCH] D113183: [LV] Patch up induction phis after VPlan execution.

Mon Nov 29 13:57:10 PST 2021

fhahn marked 2 inline comments as done.
fhahn added a comment.

In D113183#3157060 <https://reviews.llvm.org/D113183#3157060>, @Ayal wrote:

> In D113183#3153441 <https://reviews.llvm.org/D113183#3153441>, @fhahn wrote:
>
>> 
>
>
>
>> Unfortunately VPWidenIntOrFpInductionRecipe is already a multi-def in some cases (due to handling casts as well).
>
> wonder how R->addOperand(IncR->`getVPSingleValue()`); currently works in fixHeaderPhis - is a testcase of a casted induction feeding a reduction is needed?

It doesn't. I put up D114720 <https://reviews.llvm.org/D114720>  and D114739 <https://reviews.llvm.org/D114739> to fix 2 crashes in this area.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D113183