[es6] Perform the IsConstructor test in GetSuperConstructor.

src/compiler/linkage.cc

Index: src/compiler/linkage.cc
diff --git a/src/compiler/linkage.cc b/src/compiler/linkage.cc
index 643277b1d139d09752aea36b58179df69661f114..2458f65867285525783a8a6bd3dec6204421e769 100644
--- a/src/compiler/linkage.cc
+++ b/src/compiler/linkage.cc
@@ -151,7 +151,6 @@ bool Linkage::NeedsFrameStateInput(Runtime::FunctionId function) {
     case Runtime::kDefineGetterPropertyUnchecked:  // TODO(jarin): Is it safe?
     case Runtime::kDefineSetterPropertyUnchecked:  // TODO(jarin): Is it safe?
     case Runtime::kGeneratorGetContinuation:
-    case Runtime::kGetSuperConstructor:

[Henrique Ferreiro, 2016/12/08 13:18:01]

I assumed I had to remove this.

[Benedikt Meurer, 2016/12/08 18:42:24]

Just leave 'em here as long as the parser still uses it. We remove it when we
drop the runtime function completely.

[Henrique Ferreiro, 2016/12/09 13:24:28]

The runtime function throws an exception now, so it needs a FrameState, right?

[Benedikt Meurer, 2016/12/09 18:30:28]

You're absolutely right, thanks for pointing this out!

     case Runtime::kIsFunction:
     case Runtime::kNewClosure:
     case Runtime::kNewClosure_Tenured:
@@ -178,7 +177,6 @@ bool Linkage::NeedsFrameStateInput(Runtime::FunctionId function) {
     case Runtime::kInlineGeneratorClose:
     case Runtime::kInlineGeneratorGetInputOrDebugPos:
     case Runtime::kInlineGeneratorGetResumeMode:
-    case Runtime::kInlineGetSuperConstructor:
     case Runtime::kInlineIsArray:
     case Runtime::kInlineIsJSReceiver:
     case Runtime::kInlineIsRegExp: