[PATCH] D100091: [X86] Fix wrong handle with "-mno-x87"

Thu Apr 8 10:09:59 PDT 2021

craig.topper added a comment.

In D100091#2675996 <https://reviews.llvm.org/D100091#2675996>, @LiuChen3 wrote:

> In D100091#2675968 <https://reviews.llvm.org/D100091#2675968>, @craig.topper wrote:
>
>> What about cases that use x87 to do float/double conversions? I think it’s i64 conversions in 32-bit mode. But maybe other cases. Does gcc disable those with -mno-80387?
>
> You are right! GCC can do the conversions: https://godbolt.org/z/87ez838oc . 
> With this patch, llvm will fail in ISEL. I want to make another patch to do this.

I'm more concerned about these cases that don't use long double and aren't missed constant folding. https://godbolt.org/z/qGv54ef34

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D100091