[Interpreter] Generate valid FrameStates in the Bytecode Graph Builder.

src/compiler/code-generator.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 "src/compiler/code-generator.h"
 
 #include "src/address-map.h"
 #include "src/compiler/code-generator-impl.h"
 #include "src/compiler/linkage.h"
 #include "src/compiler/pipeline.h"
@@ skipping 540 matching lines @@
   }
   int shared_info_id = DefineDeoptimizationLiteral(shared_info);
 
   switch (descriptor->type()) {
     case FrameStateType::kJavaScriptFunction:
       translation->BeginJSFrame(
           descriptor->bailout_id(), shared_info_id,
           static_cast<unsigned int>(descriptor->GetSize(state_combine) -
                                     (1 + descriptor->parameters_count())));
       break;
+    case FrameStateType::kInterpretedFunction:
+      // TODO(rmcilroy): Implement interpreted function translation.

[Jarin, 2015/12/13 20:26:30]

Until this is implemented, could you have UNIMPLEMENTED or UNREACHABLE or
CHECK(false) here rather than silent happiness?

[rmcilroy, 2015/12/16 15:40:56]

This would cause all the current bytecode-graph-builder test to fail since they
would all end up reaching this point (e.g., when adding the after framestate for
calls), even if they don't trigger a deopt.

+      break;
     case FrameStateType::kArgumentsAdaptor:
       translation->BeginArgumentsAdaptorFrame(
           shared_info_id,
           static_cast<unsigned int>(descriptor->parameters_count()));
       break;
     case FrameStateType::kConstructStub:
       translation->BeginConstructStubFrame(
           shared_info_id,
           static_cast<unsigned int>(descriptor->parameters_count()));
       break;
@@ skipping 132 matching lines @@
     : frame_(gen->frame()), masm_(gen->masm()), next_(gen->ools_) {
   gen->ools_ = this;
 }
 
 
 OutOfLineCode::~OutOfLineCode() {}
 
 }  // namespace compiler
 }  // namespace internal
 }  // namespace v8