[PATCH] D88287: [NARY-REASSOCIATE] Support reassociation of min/max

Fri Apr 9 03:39:52 PDT 2021

ebrevnov added a comment.

In D88287#2673846 <https://reviews.llvm.org/D88287#2673846>, @ebrevnov wrote:

> In D88287#2672928 <https://reviews.llvm.org/D88287#2672928>, @critson wrote:
>
>> Raising this here as well, as it seems the previous concern I raised with the commit was ignored.
>
> Thanks for letting me know. Hadn't seen this before. I can reproduce the issue and working on the fix.
>
>> I am seeing this change go into an infinite loop attempting to example an expression, adding more and more .nary postfixes.
>> I have reduced the failure to the attached bugpoint.
>>
>> llc -march=amdgcn -mcpu=gfx700 < bugpoint-reduced-simplified.ll
>> F16169461: bugpoint-reduced-simplified.ll <https://reviews.llvm.org/F16169461>

Fix has been submitted for review https://reviews.llvm.org/D100170

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D88287