Reviewers: fschneider,

Description:
MIPS: Fix for function argument access in non-strict mode.

This fix resolves failures in sputnik tests: S13_A13_T1,
S13_A13_T2, S13_A13_T3. The tests check function argument after it
is deleted in non-strict mode.

BUG=
TEST=


Please review this at http://codereview.chromium.org/7740019/

Affected files:
  M src/mips/ic-mips.cc


Index: src/mips/ic-mips.cc
diff --git a/src/mips/ic-mips.cc b/src/mips/ic-mips.cc
index da39962691eab6e5275b3d5c18a2c015aaac8409..85cb9164880cbcc87ea62c5d19e1ed33f172bd1d 100644
--- a/src/mips/ic-mips.cc
+++ b/src/mips/ic-mips.cc
@@ -885,8 +885,8 @@ void KeyedLoadIC::GenerateNonStrictArguments(MacroAssembler* masm) {
   MemOperand unmapped_location =
       GenerateUnmappedArgumentsLookup(masm, a0, a2, a3, &slow);
   __ lw(a2, unmapped_location);
-  __ Branch(&slow, eq, a2, Operand(a3));
   __ LoadRoot(a3, Heap::kTheHoleValueRootIndex);
+  __ Branch(&slow, eq, a2, Operand(a3));
   __ mov(v0, a2);
   __ Ret();
   __ bind(&slow);


--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev

Reply via email to