Detect leaf optimized methods and skip stack check overflow test in those, unless it is in a loop (…

runtime/vm/intermediate_language.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/intermediate_language.h"
 
 #include "vm/bit_vector.h"
 #include "vm/dart_entry.h"
 #include "vm/flow_graph_allocator.h"
 #include "vm/flow_graph_builder.h"
@@ skipping 1493 matching lines @@
   const intptr_t kNumInputs = 1;
   const intptr_t kNumTemps = 0;
   LocationSummary* summary =
       new LocationSummary(kNumInputs, kNumTemps, LocationSummary::kNoCall);
   summary->set_in(0, Location::RegisterLocation(CTX));
   return summary;
 }
 
 
 void StoreContextInstr::EmitNativeCode(FlowGraphCompiler* compiler) {
-  // Nothing to do.  Context register were loaded by register allocator.
+  // Nothing to do. Context register were loaded by register allocator.

[Kevin Millikin (Google), 2013/04/18 10:05:28]

I like the space.  If you're going to make a change to this line, you could also
change 'were' to 'was' or 'is' and add the article 'the' before register
allocator.  :)

[srdjan, 2013/04/18 18:15:57]

Added space back, changed were to was and added the.

   ASSERT(locs()->in(0).reg() == CTX);
 }
 
 
 StrictCompareInstr::StrictCompareInstr(Token::Kind kind,
                                        Value* left,
                                        Value* right)
     : ComparisonInstr(kind, left, right),
       needs_number_check_(FLAG_new_identity_spec) {
   ASSERT((kind == Token::kEQ_STRICT) || (kind == Token::kNE_STRICT));
@@ skipping 863 matching lines @@
     default:
       UNREACHABLE();
   }
   return kPowRuntimeEntry;
 }
 
 
 #undef __
 
 }  // namespace dart