Change indexed load and store IL instructions to fit with SSA backend.

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 725 matching lines @@
   compiler->GenerateCall(token_pos(),
                          try_index(),
                          &StubCode::CallNativeCFunctionLabel(),
                          PcDescriptors::kOther,
                          locs()->stack_bitmap());
   __ popl(result);
 }
 
 
 LocationSummary* LoadIndexedComp::MakeLocationSummary() const {
+  ASSERT((receiver_type() == kGrowableObjectArrayCid) ||
+         (receiver_type() == kArrayCid) ||
+         (receiver_type() == kImmutableArrayCid));
   const intptr_t kNumInputs = 2;
-  if ((receiver_type() == kGrowableObjectArrayCid) ||
-      (receiver_type() == kArrayCid) ||
-      (receiver_type() == kImmutableArrayCid)) {
-    const intptr_t kNumTemps = 1;
-    LocationSummary* locs =
-        new LocationSummary(kNumInputs, kNumTemps, LocationSummary::kNoCall);
-    locs->set_in(0, Location::RequiresRegister());
-    locs->set_in(1, Location::RequiresRegister());
-    locs->set_temp(0, Location::RequiresRegister());
-    locs->set_out(Location::RequiresRegister());
-    return locs;
-  } else {
-    ASSERT(receiver_type() == kIllegalCid);
-    return MakeCallSummary();
-  }
-}
-
-
-static void EmitLoadIndexedPolymorphic(FlowGraphCompiler* compiler,
-                                       LoadIndexedComp* comp) {
-  Label* deopt = compiler->AddDeoptStub(comp->deopt_id(),
-                                        comp->try_index(),
-                                        kDeoptLoadIndexedPolymorphic);
-  ASSERT(comp->ic_data()->NumberOfChecks() > 0);
-  ASSERT(comp->HasICData());
-  const ICData& ic_data = *comp->ic_data();
-  ASSERT(ic_data.num_args_tested() == 1);
-  // No indexed access on Smi.
-  ASSERT(ic_data.GetReceiverClassIdAt(0) != kSmiCid);
-  // 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.
-                            comp->deopt_id(),
-                            comp->token_pos(),
-                            comp->try_index(),
-                            comp->locs()->stack_bitmap());
+  const intptr_t kNumTemps = 1;
+  LocationSummary* locs =
+      new LocationSummary(kNumInputs, kNumTemps, LocationSummary::kNoCall);
+  locs->set_in(0, Location::RequiresRegister());
+  locs->set_in(1, Location::RequiresRegister());
+  locs->set_temp(0, Location::RequiresRegister());
+  locs->set_out(Location::RequiresRegister());
+  return locs;
 }
 
 
 void LoadIndexedComp::EmitNativeCode(FlowGraphCompiler* compiler) {
-  if (receiver_type() == kIllegalCid) {
-    if (HasICData() && (ic_data()->NumberOfChecks() > 0)) {
-      EmitLoadIndexedPolymorphic(compiler, this);
-    } else {
-      compiler->EmitLoadIndexedGeneric(this);
-    }
-    ASSERT(locs()->out().reg() == EAX);
-    return;
-  }
-
   Register receiver = locs()->in(0).reg();
   Register index = locs()->in(1).reg();
   Register result = locs()->out().reg();
   Register temp = locs()->temp(0).reg();
 
   const DeoptReasonId deopt_reason =
       (receiver_type() == kGrowableObjectArrayCid) ?
       kDeoptLoadIndexedGrowableArray : kDeoptLoadIndexedFixedArray;
 
-  Label* deopt = compiler->AddDeoptStub(deopt_id(),
-                                        try_index(),
+  Label* deopt = compiler->AddDeoptStub(original()->deopt_id(),
+                                        original()->try_index(),
                                         deopt_reason,
                                         receiver,
                                         index);
 
   __ testl(receiver, Immediate(kSmiTagMask));  // Deoptimize if Smi.
   __ j(ZERO, deopt);
   __ CompareClassId(receiver, receiver_type(), temp);
   __ j(NOT_EQUAL, deopt);
 
   __ testl(index, Immediate(kSmiTagMask));
@@ skipping 23 matching lines @@
     }
 
     default:
       UNREACHABLE();
       break;
   }
 }
 
 
 LocationSummary* StoreIndexedComp::MakeLocationSummary() const {
+  ASSERT((receiver_type() == kGrowableObjectArrayCid) ||
+         (receiver_type() == kArrayCid));
   const intptr_t kNumInputs = 3;
-  if ((receiver_type() == kGrowableObjectArrayCid) ||
-      (receiver_type() == kArrayCid)) {
-    const intptr_t kNumTemps = 1;
-    LocationSummary* locs =
-        new LocationSummary(kNumInputs, kNumTemps, LocationSummary::kNoCall);
-    locs->set_in(0, Location::RequiresRegister());
-    locs->set_in(1, Location::RequiresRegister());
-    locs->set_in(2, Location::RequiresRegister());
-    locs->set_temp(0, Location::RequiresRegister());
-    locs->set_out(Location::NoLocation());
-    return locs;
-  } else {
-    ASSERT(receiver_type() == kIllegalCid);
-    return MakeCallSummary();
-  }
-}
-
-
-static void EmitStoreIndexedGeneric(FlowGraphCompiler* compiler,
-                                    StoreIndexedComp* comp) {
-  const String& function_name =
-      String::ZoneHandle(Symbols::New(Token::Str(Token::kASSIGN_INDEX)));
-
-  compiler->AddCurrentDescriptor(PcDescriptors::kDeopt,
-                                 comp->deopt_id(),
-                                 comp->token_pos(),
-                                 comp->try_index());
-
-  const intptr_t kNumArguments = 3;
-  const intptr_t kNumArgsChecked = 1;  // Type-feedback.
-  compiler->GenerateInstanceCall(comp->deopt_id(),
-                                 comp->token_pos(),
-                                 comp->try_index(),
-                                 function_name,
-                                 kNumArguments,
-                                 Array::ZoneHandle(),  // No named arguments.
-                                 kNumArgsChecked,
-                                 comp->locs()->stack_bitmap());
-}
-
-
-static void EmitStoreIndexedPolymorphic(FlowGraphCompiler* compiler,
-                                        StoreIndexedComp* comp) {
-  Label* deopt = compiler->AddDeoptStub(comp->deopt_id(),
-                                        comp->try_index(),
-                                        kDeoptStoreIndexedPolymorphic);
-  ASSERT(comp->ic_data()->NumberOfChecks() > 0);
-  ASSERT(comp->HasICData());
-  const ICData& ic_data = *comp->ic_data();
-  ASSERT(ic_data.num_args_tested() == 1);
-  // No indexed access on Smi.
-  ASSERT(ic_data.GetReceiverClassIdAt(0) != kSmiCid);
-  // Load receiver into EAX.
-  const intptr_t kNumArguments = 3;
-  __ 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.
-                            comp->deopt_id(),
-                            comp->token_pos(),
-                            comp->try_index(),
-                            comp->locs()->stack_bitmap());
+  const intptr_t kNumTemps = 1;
+  LocationSummary* locs =
+      new LocationSummary(kNumInputs, kNumTemps, LocationSummary::kNoCall);
+  locs->set_in(0, Location::RequiresRegister());
+  locs->set_in(1, Location::RequiresRegister());
+  locs->set_in(2, Location::RequiresRegister());
+  locs->set_temp(0, Location::RequiresRegister());
+  locs->set_out(Location::NoLocation());
+  return locs;
 }
 
 
 void StoreIndexedComp::EmitNativeCode(FlowGraphCompiler* compiler) {
-  if (receiver_type() == kIllegalCid) {
-    if (HasICData() && (ic_data()->NumberOfChecks() > 0)) {
-      EmitStoreIndexedPolymorphic(compiler, this);
-    } else {
-      EmitStoreIndexedGeneric(compiler, this);
-    }
-    return;
-  }
-
   Register receiver = locs()->in(0).reg();
   Register index = locs()->in(1).reg();
   Register value = locs()->in(2).reg();
   Register temp = locs()->temp(0).reg();
 
-  Label* deopt = compiler->AddDeoptStub(deopt_id(),
-                                        try_index(),
+  Label* deopt = compiler->AddDeoptStub(original()->deopt_id(),
+                                        original()->try_index(),
                                         kDeoptStoreIndexed,
                                         receiver,
                                         index,
                                         value);
 
   __ testl(receiver, Immediate(kSmiTagMask));  // Deoptimize if Smi.
   __ j(ZERO, deopt);
   __ CompareClassId(receiver, receiver_type(), temp);
   __ j(NOT_EQUAL, deopt);
 
@@ skipping 1269 matching lines @@
                                  locs()->stack_bitmap());
   __ CompareObject(EAX, compiler->bool_true());
   EmitBranchOnCondition(compiler, branch_condition);
 }
 
 }  // namespace dart
 
 #undef __
 
 #endif  // defined TARGET_ARCH_X64