Revision: 18349
Author:   [email protected]
Date:     Wed Dec 18 12:09:14 2013 UTC
Log:      Improve the fix for r18344.

Our toolchains differ in mysterious ways, especially regarding
overloading resolution, templates, etc. :-P This fix avoids a
call/return just for resolution and uses a static_cast instead.

[email protected]

Review URL: https://codereview.chromium.org/118183002
http://code.google.com/p/v8/source/detail?r=18349

Modified:
 /branches/bleeding_edge/src/assembler.cc

=======================================
--- /branches/bleeding_edge/src/assembler.cc    Wed Dec 18 11:08:10 2013 UTC
+++ /branches/bleeding_edge/src/assembler.cc    Wed Dec 18 12:09:14 2013 UTC
@@ -1388,15 +1388,11 @@
 #endif  // V8_INTERPRETED_REGEXP


-static double math_log_double(double x) {
-  return log(x);
-}
-
-
 ExternalReference ExternalReference::math_log_double_function(
     Isolate* isolate) {
+  typedef double (*d2d)(double x);
   return ExternalReference(Redirect(isolate,
-                                    FUNCTION_ADDR(math_log_double),
+                                    FUNCTION_ADDR(static_cast<d2d>(log)),
                                     BUILTIN_FP_CALL));
 }

--
--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
--- You received this message because you are subscribed to the Google Groups "v8-dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
For more options, visit https://groups.google.com/groups/opt_out.

Reply via email to