Re-optimize faster after making a pretenuring decision.

src/deoptimizer.cc

 // Copyright 2013 the V8 project authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "v8.h"
 
 #include "accessors.h"
 #include "codegen.h"
 #include "deoptimizer.h"
 #include "disasm.h"
@@ skipping 304 matching lines @@
       SharedFunctionInfo* shared = function->shared();
       function->set_code(shared->code());
 
       if (FLAG_trace_deopt) {
         CodeTracer::Scope scope(code->GetHeap()->isolate()->GetCodeTracer());
         PrintF(scope.file(), "[deoptimizer unlinked: ");
         function->PrintName(scope.file());
         PrintF(scope.file(),
                " / %" V8PRIxPTR "]\n", reinterpret_cast<intptr_t>(function));
       }
+
+      if (code->marked_for_instant_optimization()) {

[titzer, 2014/05/12 12:05:24]

This isn't the right place for this somehow. It's surprising that
DeoptimizeMarkedCode actually calls the compiler.

[Hannes Payer (out of office), 2014/05/12 16:31:19]

Not sure, it seems like the right place to express "deoptimize and re-optimize
immediately semantics". And since we just have the code object at gc time which
we can mark for deoptimization and re-optimization, it seems to be fine.

+        Compiler::Optimize(function, "marked for instant optimization");
+      }
     }
   };
 
   // Unlink all functions that refer to marked code.
   SelectedCodeUnlinker unlinker;
   VisitAllOptimizedFunctionsForContext(context, &unlinker);
 
   Isolate* isolate = context->GetHeap()->isolate();
 #ifdef DEBUG
   Code* topmost_optimized_code = NULL;
@@ skipping 3246 matching lines @@
 }
 
 
 void DeoptimizedFrameInfo::Iterate(ObjectVisitor* v) {
   v->VisitPointer(BitCast<Object**>(&function_));
   v->VisitPointers(parameters_, parameters_ + parameters_count_);
   v->VisitPointers(expression_stack_, expression_stack_ + expression_count_);
 }
 
 } }  // namespace v8::internal