[PATCH] D88287: [NARY-REASSOCIATE] Support reassociation of min/max
Bairen Yi via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
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
More information about the llvm-commits
mailing list