Remove the last remnants of the TranscendentalCache.

src/arm/full-codegen-arm.cc

Index: src/arm/full-codegen-arm.cc
diff --git a/src/arm/full-codegen-arm.cc b/src/arm/full-codegen-arm.cc
index f6d3ea36ab8ce6e0ce21759ce369e8be34db2666..e058a7a13c2c52e85eb4662c92bde492d23c8829 100644
--- a/src/arm/full-codegen-arm.cc
+++ b/src/arm/full-codegen-arm.cc
@@ -3698,13 +3698,11 @@ void FullCodeGenerator::EmitStringCompare(CallRuntime* expr) {
 
 
 void FullCodeGenerator::EmitMathLog(CallRuntime* expr) {
-  // Load the argument on the stack and call the stub.
-  TranscendentalCacheStub stub(TranscendentalCache::LOG,
-                               TranscendentalCacheStub::TAGGED);
+  // Load the argument on the stack and call the runtime function.
   ZoneList<Expression*>* args = expr->arguments();
   ASSERT(args->length() == 1);
   VisitForStackValue(args->at(0));
-  __ CallStub(&stub);
+  __ CallRuntime(Runtime::kMath_log, 1);

[Yang, 2013/12/13 13:35:39]

We could directly call %Math_log from math.js and get rid of %_Math_log. Then
again, that would mean that if Math.log is optimized by itself and not inlined
into its caller, it would be very slow.

[Sven Panne, 2013/12/13 13:59:50]

I came to the same conclusion (could be slow), furthermore sqrt is handled
similarly, so just changing this would be inconsistent. In a nutshell: I think
it is OK as it is in the CL, perhaps we might want to reconsider the whole
%_MathFoo story, but in a different CL.

   context()->Plug(r0);
 }