[PATCH] D87188: [InstCombine] Canonicalize SPF to abs intrinc

Thu Dec 17 14:03:58 PST 2020

mstorsjo added a comment.

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

> @mstorsjo i've bothered @dmajor to give this a try on some of mozilla's CI, and unfortunately it came back green..
> I don't recall if i can reproduce it locally (on vanilla test suite, i'll recheck),
> but right now it would seem that you are the only one who can reproduce this :/
>
> Were you able to make any progress on a somewhat more actionable reproducer?
>
> What would be really helpful is a standalone source that can be compiled and run (here by be locally, on linux x86_64)
> that produces different result with/without this change.

IIRC I'm only able to reproduce this for aarch64 (and possibly arm) targets unfortunately. I can make a kind of standalone testcase with one single file built from source, linked and executed along with a bigger piece of code (either from source or prebuilt object files), but it'd be for aarch64-linux (or aarch64-darwin if that helps - I haven't tried reproducing it there but I'd presume it behaves the same as I noticed the bug first on aarch64 windows).

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D87188