Support inlining and crankshaft optimization of Math.random.

src/ia32/lithium-ia32.cc

Index: src/ia32/lithium-ia32.cc
diff --git a/src/ia32/lithium-ia32.cc b/src/ia32/lithium-ia32.cc
index 6ecb29f5b9310720ed8c6515bb34a5ed18c18d3a..41de8f54e78daac285b4b8efed90894643ff73a6 100644
--- a/src/ia32/lithium-ia32.cc
+++ b/src/ia32/lithium-ia32.cc
@@ -1,4 +1,4 @@
-// Copyright 2011 the V8 project authors. All rights reserved.
+// Copyright 2012 the V8 project authors. All rights reserved.
 // Redistribution and use in source and binary forms, with or without
 // modification, are permitted provided that the following conditions are
 // met:
@@ -1469,6 +1469,17 @@ LInstruction* LChunkBuilder::DoPower(HPower* instr) {
 }
 
 
+LInstruction* LChunkBuilder::DoRandom(HRandom* instr) {
+  ASSERT(instr->representation().IsDouble());
+  ASSERT(instr->global_object()->representation().IsTagged());
+  LOperand* global_object = UseFixed(instr->global_object(), eax);
+  LRandom* result = new(zone()) LRandom(global_object);
+  return MarkAsCall(DefineFixedDouble(result, xmm1),
+                    instr,
+                    CAN_DEOPTIMIZE_EAGERLY);

[fschneider, 2012/01/11 09:16:28]

Since you don't have a DeoptimizeIf in the generated code, you  need
CANNOT_DEOPTIMIZE_EAGERLY here (which is the default).

[Mads Ager (chromium), 2012/01/11 10:24:46]

Good catch! Thanks Florian. Done.

+}
+
+
 LInstruction* LChunkBuilder::DoCompareGeneric(HCompareGeneric* instr) {
   ASSERT(instr->left()->representation().IsTagged());
   ASSERT(instr->right()->representation().IsTagged());