For trigonometric functions call to C-libraries: they are faster than x87 operations (3% on Box2D).

runtime/vm/intermediate_language_x64.cc

 // Copyright (c) 2013, the Dart project authors.  Please see the AUTHORS file
 // for details. All rights reserved. Use of this source code is governed by a
 // BSD-style license that can be found in the LICENSE file.
 
 #include "vm/globals.h"  // Needed here to get TARGET_ARCH_X64.
 #if defined(TARGET_ARCH_X64)
 
 #include "vm/intermediate_language.h"
 
 #include "vm/dart_entry.h"
@@ skipping 3835 matching lines @@
       break;
     case Token::kSUB:
       __ subpl(left, right);
       break;
     default: UNREACHABLE();
   }
 }
 
 
 LocationSummary* MathUnaryInstr::MakeLocationSummary() const {
+  if ((kind() == MethodRecognizer::kMathSin) ||
+      (kind() == MethodRecognizer::kMathCos)) {
+    // Calling convention on x64 uses XMM0 and XMM1 to pass the first two
+    // double arguments and XMM0 to return the result. Unfortunately
+    // currently we can't specify these registers because ParallelMoveResolver
+    // assumes that XMM0 is free at all times.
+    // TODO(vegorov): allow XMM0 to be used.

[regis, 2013/09/06 18:45:08]

For the same reason, we use Q7 instead of Q0 as TMP on ARM.

+    const intptr_t kNumTemps = 0;
+    LocationSummary* summary =
+        new LocationSummary(InputCount(), kNumTemps, LocationSummary::kCall);
+    summary->set_in(0, Location::FpuRegisterLocation(XMM1));
+    summary->set_out(Location::FpuRegisterLocation(XMM1));
+    return summary;
+  }
   const intptr_t kNumInputs = 1;
   const intptr_t kNumTemps = 0;
   LocationSummary* summary =
       new LocationSummary(kNumInputs, kNumTemps, LocationSummary::kNoCall);
   summary->set_in(0, Location::RequiresFpuRegister());
   summary->set_out(Location::RequiresFpuRegister());
   return summary;
 }
 
 
 void MathUnaryInstr::EmitNativeCode(FlowGraphCompiler* compiler) {
   if (kind() == MethodRecognizer::kMathSqrt) {
     __ sqrtsd(locs()->out().fpu_reg(), locs()->in(0).fpu_reg());
-  } else if ((kind() == MethodRecognizer::kMathCos) ||
-             (kind() == MethodRecognizer::kMathSin)) {
-    __ pushq(RAX);
-    __ movsd(Address(RSP, 0), locs()->in(0).fpu_reg());
-    __ fldl(Address(RSP, 0));
-    if (kind() == MethodRecognizer::kMathSin) {
-      __ fsin();
-    } else {
-      ASSERT(kind() == MethodRecognizer::kMathCos);
-      __ fcos();
-    }
-    __ fstpl(Address(RSP, 0));
-    __ movsd(locs()->out().fpu_reg(), Address(RSP, 0));
-    __ addq(RSP, Immediate(kWordSize));
-    return;
   } else {
-    UNREACHABLE();
+    __ EnterFrame(0);
+    __ ReserveAlignedFrameSpace(0);
+    __ movaps(XMM0, locs()->in(0).fpu_reg());
+    __ CallRuntime(TargetFunction(), InputCount());
+    __ movaps(locs()->out().fpu_reg(), XMM0);
+    __ leave();
   }
 }
 
 
 LocationSummary* UnarySmiOpInstr::MakeLocationSummary() const {
   const intptr_t kNumInputs = 1;
   return LocationSummary::Make(kNumInputs,
                                Location::SameAsFirstInput(),
                                LocationSummary::kNoCall);
 }
@@ skipping 910 matching lines @@
                          PcDescriptors::kOther,
                          locs());
   __ Drop(2);  // Discard type arguments and receiver.
 }
 
 }  // namespace dart
 
 #undef __
 
 #endif  // defined TARGET_ARCH_X64