[PATCH] D99294: [VPlan] Representing backedge def-use feeding reduction phis.

Florian Hahn via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue May 11 06:35:26 PDT 2021


fhahn added a comment.

In D99294#2750159 <https://reviews.llvm.org/D99294#2750159>, @uabelho wrote:

> Hi,
>
> I wrote a PR about a LV-crash that started happening with this patch:
>  https://bugs.llvm.org/show_bug.cgi?id=50298

Thanks for the report! The problem was that we did not record the recipe if a simplified VPValue is used. I pushed faebc6bf108e <https://reviews.llvm.org/rGfaebc6bf108eccdfd75917636c64137f73a7bda7> to fix it for now, but as Ayal pointed out, it might be worth the simplify the handling, which I'll look into separately.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D99294



More information about the llvm-commits mailing list