[PATCH] D128755: [VPlan] Make sure optimizeInductions removes wide ind from scalar plan.

Wed Jun 29 09:18:43 PDT 2022

bmahjour added a comment.

In D128755#3618609 <https://reviews.llvm.org/D128755#3618609>, @fhahn wrote:

> Address latest comments, thanks!
>
> In D128755#3617144 <https://reviews.llvm.org/D128755#3617144>, @bmahjour wrote:
>
>> could you please add the crashing IR from https://reviews.llvm.org/D123720 as an additional test? It still seems to crash with this patch!
>
> Is it still crashing with this patch applied to current `main`? It needs b18141a8f29f638be0602aa6ffbfc2d434b0b74a <https://reviews.llvm.org/rGb18141a8f29f638be0602aa6ffbfc2d434b0b74a> as well.

I was indeed missing b18141a8f29f638be0602aa6ffbfc2d434b0b74a <https://reviews.llvm.org/rGb18141a8f29f638be0602aa6ffbfc2d434b0b74a>. I've verified that the crash is gone with this patch on top of latest trunk. Thank you!

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D128755