Experimental: when doing memory-to-memory moves as part of a frame...

src/codegen-ia32.cc

 // Copyright 2006-2008 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:
 //
 //     * Redistributions of source code must retain the above copyright
 //       notice, this list of conditions and the following disclaimer.
 //     * Redistributions in binary form must reproduce the above
 //       copyright notice, this list of conditions and the following
 //       disclaimer in the documentation and/or other materials provided
@@ skipping 172 matching lines @@
       Result answer = frame_->CallStub(&stub, 3);
       frame_->Push(&answer);
     }
 
     if (scope_->num_heap_slots() > 0) {
       Comment cmnt(masm_, "[ allocate local context");
       // Allocate local context.
       // Get outer context and create a new context based on it.
       frame_->PushFunction();
       Result context = frame_->CallRuntime(Runtime::kNewContext, 1);
+      // Update context local.
+      frame_->SaveContextRegister();
 
       if (kDebug) {
         JumpTarget verified_true(this);
         // Verify eax and esi are the same in debug mode
         __ cmp(context.reg(), Operand(esi));
         context.Unuse();
         verified_true.Branch(equal);
         frame_->SpillAll();
         __ int3();
         verified_true.Bind();
       }
-      // Update context local.
-      frame_->SaveContextRegister();
     }
 
     // TODO(1241774): Improve this code:
     // 1) only needed if we have a context
     // 2) no need to recompute context ptr every single time
     // 3) don't copy parameter operand code from SlotOperand!
     {
       Comment cmnt2(masm_, "[ copy context parameters into .context");
 
       // Note that iteration order is relevant here! If we have the same
@@ skipping 1549 matching lines @@
   Comment cmnt(masm_, "[ WithEnterStatement");
   CodeForStatementPosition(node);
   Load(node->expression());
   Result context(this);
   if (node->is_catch_block()) {
     context = frame_->CallRuntime(Runtime::kPushCatchContext, 1);
   } else {
     context = frame_->CallRuntime(Runtime::kPushContext, 1);
   }
 
+  // Update context local.
+  frame_->SaveContextRegister();
+
   if (kDebug) {
     JumpTarget verified_true(this);
     // Verify that the result of the runtime call and the esi register are
     // the same in debug mode.
     __ cmp(context.reg(), Operand(esi));
     context.Unuse();
     verified_true.Branch(equal);
     frame_->SpillAll();
     __ int3();
     verified_true.Bind();
   }
-
-  // Update context local.
-  frame_->SaveContextRegister();
 }
 
 
 void CodeGenerator::VisitWithExitStatement(WithExitStatement* node) {
   ASSERT(!in_spilled_code());
   Comment cmnt(masm_, "[ WithExitStatement");
   CodeForStatementPosition(node);
   // Pop context.
   __ mov(esi, ContextOperand(esi, Context::PREVIOUS_INDEX));
   // Update context local.
@@ skipping 4551 matching lines @@
 
   // Slow-case: Go through the JavaScript implementation.
   __ bind(&slow);
   __ InvokeBuiltin(Builtins::INSTANCE_OF, JUMP_FUNCTION);
 }
 
 
 #undef __
 
 } }  // namespace v8::internal