[interpreter] CompilationInfo::unoptimized_code only for OSR.

src/compiler.h

 // Copyright 2012 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.
 
 #ifndef V8_COMPILER_H_
 #define V8_COMPILER_H_
 
 #include "src/allocation.h"
 #include "src/ast/ast.h"
 #include "src/bailout-reason.h"
@@ skipping 229 matching lines @@
 
   bool has_global_object() const { return has_native_context(); }
 
   JSGlobalObject* global_object() const {
     return has_global_object() ? native_context()->global_object() : nullptr;
   }
 
   // Accessors for the different compilation modes.
   bool IsOptimizing() const { return mode_ == OPTIMIZE; }
   bool IsStub() const { return mode_ == STUB; }
-  void SetOptimizing(BailoutId osr_ast_id, Handle<Code> unoptimized) {
+  void SetOptimizing(BailoutId osr_ast_id = BailoutId::None(),
+                     Handle<Code> unoptimized = Handle<Code>::null()) {

[rmcilroy, 2016/02/09 22:06:07]

nit - maybe remove these parameters, and instead add a SetOptimizingForOsr
function rather than adding defaults?

[Michael Starzinger, 2016/02/10 10:01:11]

Done. Nice idea, I like!

     DCHECK(has_shared_info());
     SetMode(OPTIMIZE);
     osr_ast_id_ = osr_ast_id;
     unoptimized_code_ = unoptimized;
     optimization_id_ = isolate()->NextOptimizationId();
     code_flags_ =
         Code::KindField::update(code_flags_, Code::OPTIMIZED_FUNCTION);
   }
 
   // Deoptimization support.
@@ skipping 401 matching lines @@
   size_t info_zone_start_allocation_size_;
   base::ElapsedTimer timer_;
 
   DISALLOW_COPY_AND_ASSIGN(CompilationPhase);
 };
 
 }  // namespace internal
 }  // namespace v8
 
 #endif  // V8_COMPILER_H_