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

Thu Mar 11 22:25:31 PST 2021

byronyi added a comment.

In D88287#2621319 <https://reviews.llvm.org/D88287#2621319>, @lebedev.ri wrote:

> In D88287#2621211 <https://reviews.llvm.org/D88287#2621211>, @ebrevnov wrote:
>
>> In D88287#2619803 <https://reviews.llvm.org/D88287#2619803>, @byronyi wrote:
>>
>>> This commit may cause multiple CI failures on TensorFlow/NVPTX backend. See https://github.com/tensorflow/tensorflow/commit/b1758bd553dfc2ebbfd07eec01d1e3254eda25b8#commitcomment-48080097.
>>>
>>> I am trying to find a minimal reproducible test case here.
>>
>> According to bisect log the guilty commit is "first bad commit: [b46c085d2b6d15873fb53718f0a70b3848e19e4a <https://reviews.llvm.org/rGb46c085d2b6d15873fb53718f0a70b3848e19e4a>] [NFCI] SCEVExpander: emit intrinsics for integral {u,s}{min,max} SCEV expressions". Am I missing something?
>
> Right now this seems like a downstream problem. I've yet to see a reproducer.

Sorry, posted to the wrong venue. Downstream seems to get a workaround and I will test again and get back to you.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D88287