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

Craig Topper via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Aug 4 17:10:18 PDT 2017


craig.topper created this revision.

Previously we would use these instructions if sse was disabled and fastmath was enabled.

As mentioned in https://reviews.llvm.org/D28335, this is a bad idea.


https://reviews.llvm.org/D36344

Files:
  lib/Target/X86/X86ISelLowering.cpp
  test/CodeGen/X86/sincos-opt.ll
  test/CodeGen/X86/sincos.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D36344.109839.patch
Type: text/x-patch
Size: 8263 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170805/e115b612/attachment.bin>


More information about the llvm-commits mailing list