[LLVMbugs] [Bug 19314] New: [ARM64] Use of __sincos_stret on non-Darwin platforms.

[bugzilla-daemon at llvm.org]

http://llvm.org/bugs/show_bug.cgi?id=19314

            Bug ID: 19314
           Summary: [ARM64] Use of __sincos_stret on non-Darwin platforms.
           Product: tools
           Version: trunk
          Hardware: PC
                OS: Linux
            Status: NEW
          Severity: normal
          Priority: P
         Component: llc
          Assignee: unassignedbugs at nondot.org
          Reporter: mcrosier at codeaurora.org
                CC: apazos at codeaurora.org, bob.wilson at apple.com,
                    grosbach at apple.com, llvmbugs at cs.uiuc.edu,
                    t.p.northover at gmail.com
    Classification: Unclassified

>From ARM64ISelLowering.cpp:
-----------------------------------------------------------------------
// For iOS, we don't want to the normal expansion of a libcall to
// sincos. We want to issue a libcall to __sincos_stret to avoid memory
// traffic.
   setOperationAction(ISD::FSINCOS, MVT::f64, Custom);
   setOperationAction(ISD::FSINCOS, MVT::f32, Custom);
-----------------------------------------------------------------------

This code needs to be conditionalized to accommodate non-Darwin platforms. 
This is causing link-time failures for a few SPEC benchmarks when using the
–fno-math-errno flag, which is required for peak performance.

I believe it suffices to just expand these on non-Darwin platforms:
-  setOperationAction(ISD::FSINCOS, MVT::f64, Custom);
-  setOperationAction(ISD::FSINCOS, MVT::f32, Custom);
+  setOperationAction(ISD::FSINCOS, MVT::f64, Expand);
+  setOperationAction(ISD::FSINCOS, MVT::f32, Expand);

-- 
You are receiving this mail because:
You are on the CC list for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-bugs/attachments/20140402/2ddd841e/attachment.html>