Check all cases with "__ movl(FieldAddress" and converted them to StoreIntoObject where appropriate.

vm/code_generator_ia32.cc

 // Copyright (c) 2011, the Dart project authors.  Please see the AUTHORS file
 // for details. All rights reserved. Use of this source code is governed by a
 // BSD-style license that can be found in the LICENSE file.
 
 #include "vm/globals.h"  // Needed here to get TARGET_ARCH_IA32.
 #if defined(TARGET_ARCH_IA32)
 
 #include "vm/code_generator.h"
 
 #include "lib/error.h"
@@ skipping 280 matching lines @@
     __ nop(5);
     const Function& function =
         Function::ZoneHandle(parsed_function_.function().raw());
     __ LoadObject(EAX, function);
     __ movl(EBX, FieldAddress(EAX, Function::invocation_counter_offset()));
     __ incl(EBX);
     if (may_optimize) {
       __ cmpl(EBX, Immediate(FLAG_optimization_invocation_threshold));
       __ j(GREATER, &StubCode::OptimizeInvokedFunctionLabel());
     }
+    // EBX is an integer value (not an object).
     __ movl(FieldAddress(EAX, Function::invocation_counter_offset()), EBX);
   }
 }
 
 
 // Verify assumptions (in debug mode only).
 // - No two deopt descriptors have the same node id (deoptimization).
 // - No two ic-call descriptors have the same node id (type feedback).
 // - No two descriptors of same kind have the same PC.
 // A function without unique ids is marked as non-optimizable (e.g., because of
@@ skipping 84 matching lines @@
                                           Register scratch) {
   if (variable.is_captured()) {
     // The variable lives in the context.
     int delta = state()->context_level() - variable.owner()->context_level();
     ASSERT(delta >= 0);
     Register base = CTX;
     while (delta-- > 0) {
       __ movl(scratch, FieldAddress(base, Context::parent_offset()));
       base = scratch;
     }
-    __ movl(FieldAddress(base, Context::variable_offset(variable.index())),
-            src);
+    __ StoreIntoObject(
+        base,
+        FieldAddress(base, Context::variable_offset(variable.index())),
+        src);
   } else {
     // The variable lives in the current stack frame.
     __ movl(Address(EBP, variable.index() * kWordSize), src);
   }
 }
 
 
 void CodeGenerator::GeneratePushVariable(const LocalVariable& variable,
                                          Register scratch) {
   if (variable.is_captured()) {
@@ skipping 511 matching lines @@
       (scope != NULL) ? scope->num_context_variables() : 0;
   if (num_context_variables > 0) {
     // The loop local scope declares variables that are captured.
     // Allocate and chain a new context.
     __ movl(EDX, Immediate(num_context_variables));
     const ExternalLabel label("alloc_context",
                               StubCode::AllocateContextEntryPoint());
     GenerateCall(node_sequence->token_index(), &label);
 
     // Chain the new context in EAX to its parent in CTX.
-    __ movl(FieldAddress(EAX, Context::parent_offset()), CTX);
+    __ StoreIntoObject(EAX, FieldAddress(EAX, Context::parent_offset()), CTX);
     // Set new context as current context.
     __ movl(CTX, EAX);
     state()->set_context_level(scope->context_level());
 
     // If this node_sequence is the body of the function being compiled, copy
     // the captured parameters from the frame into the context.
     if (node_sequence == parsed_function_.node_sequence()) {
       ASSERT(scope->context_level() == 1);
       const Immediate raw_null =
           Immediate(reinterpret_cast<intptr_t>(Object::null()));
@@ skipping 951 matching lines @@
   const Function& function =
       Function::ZoneHandle(parsed_function_.function().raw());
   __ LoadObject(EAX, function);
   __ movl(EBX, FieldAddress(EAX, Function::invocation_counter_offset()));
   __ incl(EBX);
   if (!FLAG_report_invocation_count) {
     // Prevent overflow.
     __ cmpl(EBX, Immediate(FLAG_optimization_invocation_threshold));
     __ j(GREATER, &done);
   }
+  // EBX is an integer value (not an object).
   __ movl(FieldAddress(EAX, Function::invocation_counter_offset()), EBX);
   __ Bind(&done);
 }
 
 
 void CodeGenerator::VisitWhileNode(WhileNode* node) {
   const Bool& bool_true = Bool::ZoneHandle(Bool::True());
   SourceLabel* label = node->label();
   __ Bind(label->continue_label());
   node->condition()->Visit(this);
@@ skipping 877 matching lines @@
                         message_buffer, kMessageBufferSize,
                         format, args);
   va_end(args);
   Isolate::Current()->long_jump_base()->Jump(1, message_buffer);
   UNREACHABLE();
 }
 
 }  // namespace dart
 
 #endif  // defined TARGET_ARCH_IA32