[PATCH] D21431: [arm+x86] Make GNU variants behave like GNU w.r.t combining sin+cos into sincos.

Daniel Sanders via llvm-commits llvm-commits at lists.llvm.org
Thu Jun 16 06:55:58 PDT 2016


dsanders created this revision.
dsanders added subscribers: rengolin, llvm-commits.
Herald added a subscriber: aemerson.

canCombineSinCosLibcall() would previously combine sin+cos into sincos for
GNUX32/GNUEABI/GNUEABIHF regardless of whether UnsafeFPMath were set or not.
However, GNU would only combine them for UnsafeFPMath because sincos does not
set errno like sin and cos do. It seems likely that this is an oversight.

http://reviews.llvm.org/D21431

Files:
  include/llvm/ADT/Triple.h
  lib/CodeGen/SelectionDAG/LegalizeDAG.cpp
  lib/CodeGen/TargetLoweringBase.cpp
  test/CodeGen/ARM/sincos.ll
  test/CodeGen/X86/sincos-opt.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D21431.60967.patch
Type: text/x-patch
Size: 5329 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20160616/681de5d8/attachment.bin>


More information about the llvm-commits mailing list