Elide unnecessary context reload in generated stubs.

src/lithium-allocator-inl.h

 // Copyright 2011 the V8 project authors. All rights reserved.
 // Redistribution and use in source and binary forms, with or without
 // modification, are permitted provided that the following conditions are
 // met:
 //
 //     * Redistributions of source code must retain the above copyright
 //       notice, this list of conditions and the following disclaimer.
 //     * Redistributions in binary form must reproduce the above
 //       copyright notice, this list of conditions and the following
 //       disclaimer in the documentation and/or other materials provided
@@ skipping 92 matching lines @@
 }
 
 
 void InputIterator::Advance() {
   ++current_;
   SkipUninteresting();
 }
 
 
 void InputIterator::SkipUninteresting() {
-  while (current_ < limit_ && instr_->InputAt(current_)->IsConstantOperand()) {
+  LOperand* current;
+  while (current_ < limit_ &&
+         (current = instr_->InputAt(current_)) != NULL &&

[ulan, 2012/12/28 15:40:49]

Looks like we can make NULL operands uninteresting by changing it to ((current =
instr_->InputAt(current_)) == NULL || current->IsConstantOperand()).
Then we can avoid if (it.Current() == NULL) checks in the .cc file.

[danno, 2012/12/28 17:38:41]

Done.

+         current->IsConstantOperand()) {
     ++current_;
   }
 }
 
 
 UseIterator::UseIterator(LInstruction* instr)
     : input_iterator_(instr), env_iterator_(instr->environment()) { }
 
 
 bool UseIterator::Done() {
@@ skipping 11 matching lines @@
 
 void UseIterator::Advance() {
   input_iterator_.Done()
       ? env_iterator_.Advance()
       : input_iterator_.Advance();
 }
 
 } }  // namespace v8::internal
 
 #endif  // V8_LITHIUM_ALLOCATOR_INL_H_