HomePhabricator

[NARY-REASSOCIATE] Support reassociation of min/max
Concern Raised83d134c3c422

Authored by Evgeniy Brevnov <ybrevnov@azul.com> on Feb 24 2021, 3:10 AM.

Description

[NARY-REASSOCIATE] Support reassociation of min/max

Support reassociation for min/max. With that we should be able to transform min(min(a, b), c) -> min(min(a, c), b) if min(a, c) is already available.

Reviewed By: mkazantsev

Differential Revision: https://reviews.llvm.org/D88287

Event Timeline

critson raised a concern with this commit.Feb 26 2021, 12:49 AM
critson added a subscriber: critson.

This code goes into an infinite loop on AMDGPU backend.
See attached bugpoint test case:
llc-march=amdgcn -mcpu=gfx700 < bugpoint-reduced-simplified.ll

This commit now has outstanding concerns.Feb 26 2021, 12:49 AM