Replace InstanceSetterComp instruction with a plain instance call.

vm/intermediate_language_ia32.cc

 // Copyright (c) 2012, 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/intermediate_language.h"
 
 #include "lib/error.h"
@@ skipping 931 matching lines @@
       break;
     }
 
     default:
       UNREACHABLE();
       break;
   }
 }
 
 
-LocationSummary* InstanceSetterComp::MakeLocationSummary() const {
-  return MakeCallSummary();
-}
-
-
-void InstanceSetterComp::EmitNativeCode(FlowGraphCompiler* compiler) {
-  Label* deopt = NULL;
-  if (compiler->is_optimizing()) {
-    deopt = compiler->AddDeoptStub(cid(),
-                                   token_pos(),
-                                   try_index(),
-                                   kDeoptInstanceSetter);
-  }
-  if (HasICData() && (ic_data()->NumberOfChecks() > 0)) {
-    // No index-setter on Smi's.
-    ASSERT(ic_data()->GetReceiverClassIdAt(0) != kSmi);
-    // Load receiver into EAX.
-    const intptr_t kNumArguments = 2;
-    __ movl(EAX, Address(ESP, (kNumArguments - 1) * kWordSize));
-    __ testl(EAX, Immediate(kSmiTagMask));
-    __ j(ZERO, deopt);
-    __ LoadClassId(EDI, EAX);
-    compiler->EmitTestAndCall(*ic_data(),
-                              EDI,  // Class id register.
-                              kNumArguments,
-                              Array::Handle(),  // No named arguments.
-                              deopt,  // Deoptimize target.
-                              NULL,   // Fallthrough when done.
-                              cid(),
-                              token_pos(),
-                              try_index());
-
-  } else if (compiler->is_optimizing()) {
-    // Get some IC data then optimize again.
-    __ jmp(deopt);
-  } else {
-    // Unoptimized code.
-    const String& function_name =
-        String::ZoneHandle(Field::SetterSymbol(field_name()));
-
-    compiler->AddCurrentDescriptor(PcDescriptors::kDeopt,
-                                   cid(),
-                                   token_pos(),
-                                   try_index());
-    const intptr_t kArgumentCount = 2;
-    const intptr_t kCheckedArgumentCount = 1;
-    compiler->GenerateInstanceCall(cid(),
-                                   token_pos(),
-                                   try_index(),
-                                   function_name,
-                                   kArgumentCount,
-                                   Array::ZoneHandle(),
-                                   kCheckedArgumentCount);
-  }
-}
-
-
 LocationSummary* LoadInstanceFieldComp::MakeLocationSummary() const {
   // TODO(fschneider): For this instruction the input register may be
   // reused for the result (but is not required to) because the input
   // is not used after the result is defined.  We should consider adding
   // this information to the input policy.
   return LocationSummary::Make(1,
                                Location::RequiresRegister(),
                                LocationSummary::kNoCall);
 }
 
@@ skipping 1148 matching lines @@
   ASSERT(locs()->out().reg() == EAX);
   __ CompareObject(locs()->out().reg(), compiler->bool_true());
   EmitBranchOnCondition(compiler, branch_condition);
 }
 
 }  // namespace dart
 
 #undef __
 
 #endif  // defined TARGET_ARCH_X64