[PATCH] D36344: [X86] Don't use fsin/fcos/fsincos instructions ever

[Craig Topper via Phabricator via llvm-commits]

craig.topper added inline comments.


================
Comment at: test/CodeGen/X86/sincos-opt.ll:124
+; GNU_SINCOS_FASTMATH: fldt 32(%{{[re]}}sp)
 ; GNU_SINCOS_FASTMATH: faddp %st(1)
   %call = tail call x86_fp80 @sinl(x86_fp80 %x) readnone
----------------
DavidKreitzer wrote:
> Was moving the check for 'ret' here intentional?
I think all I did was copy the GNU_SINCOS check lines from above and changed them to say GNU_SINCOS_FASTMATH. I hadn't noticed the difference in the ret.


https://reviews.llvm.org/D36344