Strict mode delete.

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 2685fcb735cee0501a1d20c5c5231dc75375fdd6..932ccfaf382aebe1427ebb9f026adaae2f211ca7 100644
--- a/src/arm/full-codegen-arm.cc
+++ b/src/arm/full-codegen-arm.cc
@@ -3063,13 +3063,17 @@ void FullCodeGenerator::VisitUnaryOperation(UnaryOperation* expr) {
         } else {
           VisitForStackValue(prop->obj());
           VisitForStackValue(prop->key());
+          __ mov(r1, Operand(Smi::FromInt(strict_mode_flag())));
+          __ push(r1);

[Martin Maly, 2011/02/14 05:15:22]

Is there a preferred register to use instead?

[Mads Ager (chromium), 2011/02/14 10:27:24]

r1 is fine, r0 would be fine too. No preferences here. The only reason why the
register choice would matter here is if you need to push more than one value at
which point you can use Push to use the stm (store multiple) instruction. Here
it is clearer to just visit for stack value for the two first parameters and
just push the last one as you are doing.

           __ InvokeBuiltin(Builtins::DELETE, CALL_JS);
           context()->Plug(r0);
         }
       } else if (var->is_global()) {
-        __ ldr(r1, GlobalObjectOperand());
-        __ mov(r0, Operand(var->name()));
-        __ Push(r1, r0);
+        ASSERT(strict_mode_flag() == kNonStrictMode);

[Mads Ager (chromium), 2011/02/14 10:27:24]

Maybe duplicate the comment here to explain the assert?

[Martin Maly, 2011/02/14 21:46:51]

Done.

+        __ ldr(r2, GlobalObjectOperand());
+        __ mov(r1, Operand(var->name()));
+        __ mov(r0, Operand(Smi::FromInt(kNonStrictMode)));
+        __ Push(r2, r1, r0);
         __ InvokeBuiltin(Builtins::DELETE, CALL_JS);
         context()->Plug(r0);
       } else {