VM: Improve unoptimized code size by avoid emitting field guards.

runtime/vm/intermediate_language_arm.cc

 // Copyright (c) 2013, 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_ARM.
 #if defined(TARGET_ARCH_ARM)
 
 #include "vm/intermediate_language.h"
 
 #include "vm/cpu.h"
@@ skipping 1562 matching lines @@
 
   return summary;
 }
 
 
 void GuardFieldClassInstr::EmitNativeCode(FlowGraphCompiler* compiler) {
   const intptr_t value_cid = value()->Type()->ToCid();
   const intptr_t field_cid = field().guarded_cid();
   const intptr_t nullability = field().is_nullable() ? kNullCid : kIllegalCid;
 
-  if (field_cid == kDynamicCid) {
-    ASSERT(!compiler->is_optimizing());
-    return;  // Nothing to emit.
-  }
+  ASSERT(field_cid != kDynamicCid);
 
   const bool emit_full_guard =
       !compiler->is_optimizing() || (field_cid == kIllegalCid);
 
   const bool needs_value_cid_temp_reg = emit_full_guard ||
       ((value_cid == kDynamicCid) && (field_cid != kSmiCid));
 
   const bool needs_field_temp_reg = emit_full_guard;
 
   const Register value_reg = locs()->in(0).reg();
@@ skipping 131 matching lines @@
         zone, kNumInputs, kNumTemps, LocationSummary::kNoCall);
     summary->set_in(0, Location::RequiresRegister());
     summary->set_temp(0, Location::RequiresRegister());
     return summary;
   }
   UNREACHABLE();
 }
 
 
 void GuardFieldLengthInstr::EmitNativeCode(FlowGraphCompiler* compiler) {
-  if (field().guarded_list_length() == Field::kNoFixedLength) {
-    ASSERT(!compiler->is_optimizing());
-    return;  // Nothing to emit.
-  }
+  ASSERT(field().guarded_list_length() != Field::kNoFixedLength);
 
   Label* deopt = compiler->is_optimizing() ?
       compiler->AddDeoptStub(deopt_id(), ICData::kDeoptGuardField) : NULL;
 
   const Register value_reg = locs()->in(0).reg();
 
   if (!compiler->is_optimizing() ||
       (field().guarded_list_length() == Field::kUnknownFixedLength)) {
     const Register field_reg = locs()->temp(0).reg();
     const Register offset_reg = locs()->temp(1).reg();
@@ skipping 5131 matching lines @@
                                 1,
                                 locs());
   __ Drop(1);
   __ Pop(result);
 }
 
 
 }  // namespace dart
 
 #endif  // defined TARGET_ARCH_ARM