Revision: 18960
Author:   [email protected]
Date:     Thu Jan 30 18:13:38 2014 UTC
Log:      MIPS: Optimize HWrapReceiver.

Port r18945 (699b03e)

BUG=
[email protected]

Review URL: https://codereview.chromium.org/145083018

Patch from Balazs Kilvady <[email protected]>.
http://code.google.com/p/v8/source/detail?r=18960

Modified:
 /branches/bleeding_edge/src/mips/lithium-codegen-mips.cc
 /branches/bleeding_edge/src/mips/lithium-mips.h

=======================================
--- /branches/bleeding_edge/src/mips/lithium-codegen-mips.cc Wed Jan 29 21:51:08 2014 UTC +++ /branches/bleeding_edge/src/mips/lithium-codegen-mips.cc Thu Jan 30 18:13:38 2014 UTC
@@ -3304,19 +3304,21 @@
   // passed unchanged to builtins and strict-mode functions.
   Label global_object, result_in_receiver;

-  // Do not transform the receiver to object for strict mode
-  // functions.
-  __ lw(scratch,
-         FieldMemOperand(function, JSFunction::kSharedFunctionInfoOffset));
-  __ lw(scratch,
- FieldMemOperand(scratch, SharedFunctionInfo::kCompilerHintsOffset));
+  if (!instr->hydrogen()->known_function()) {
+    // Do not transform the receiver to object for strict mode
+    // functions.
+    __ lw(scratch,
+ FieldMemOperand(function, JSFunction::kSharedFunctionInfoOffset));
+    __ lw(scratch,
+ FieldMemOperand(scratch, SharedFunctionInfo::kCompilerHintsOffset));

-  // Do not transform the receiver to object for builtins.
-  int32_t strict_mode_function_mask =
- 1 << (SharedFunctionInfo::kStrictModeFunction + kSmiTagSize);
-  int32_t native_mask = 1 << (SharedFunctionInfo::kNative + kSmiTagSize);
- __ And(scratch, scratch, Operand(strict_mode_function_mask | native_mask));
-  __ Branch(&result_in_receiver, ne, scratch, Operand(zero_reg));
+    // Do not transform the receiver to object for builtins.
+    int32_t strict_mode_function_mask =
+        1 <<  (SharedFunctionInfo::kStrictModeFunction + kSmiTagSize);
+    int32_t native_mask = 1 << (SharedFunctionInfo::kNative + kSmiTagSize);
+ __ And(scratch, scratch, Operand(strict_mode_function_mask | native_mask));
+    __ Branch(&result_in_receiver, ne, scratch, Operand(zero_reg));
+  }

   // Normal function. Replace undefined or null with global receiver.
   __ LoadRoot(scratch, Heap::kNullValueRootIndex);
@@ -3331,8 +3333,8 @@
   __ GetObjectType(receiver, scratch, scratch);
   DeoptimizeIf(lt, instr->environment(),
                scratch, Operand(FIRST_SPEC_OBJECT_TYPE));
+
   __ Branch(&result_in_receiver);
-
   __ bind(&global_object);
   __ lw(result, FieldMemOperand(function, JSFunction::kContextOffset));
   __ lw(result,
=======================================
--- /branches/bleeding_edge/src/mips/lithium-mips.h Wed Jan 29 14:03:32 2014 UTC +++ /branches/bleeding_edge/src/mips/lithium-mips.h Thu Jan 30 18:13:38 2014 UTC
@@ -548,6 +548,7 @@
   }

   DECLARE_CONCRETE_INSTRUCTION(WrapReceiver, "wrap-receiver")
+  DECLARE_HYDROGEN_ACCESSOR(WrapReceiver)

   LOperand* receiver() { return inputs_[0]; }
   LOperand* function() { return inputs_[1]; }

--
--
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