Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(120)

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: src/compiler/verifier.cc

Issue 2504553003: [es6] Perform the IsConstructor test in GetSuperConstructor. (Closed)
Patch Set: Fix assembler and add unittest Created 4 years ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« src/compiler/typer.cc ('K') | « src/compiler/typer.cc ('k') | src/interpreter/bytecode-array-builder.h » ('j') | src/interpreter/interpreter.cc » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: src/compiler/verifier.cc
diff --git a/src/compiler/verifier.cc b/src/compiler/verifier.cc
index dbe41a665ebd6136005698b92fd998afef8a3176..7a5dd21417a5174559bd7b03d22ed0f898722566 100644
--- a/src/compiler/verifier.cc
+++ b/src/compiler/verifier.cc
@@ -635,6 +635,11 @@ void Verifier::Visitor::Check(Node* node) {
// Type is String.
CheckTypeIs(node, Type::String());
break;
+ case IrOpcode::kJSGetSuperConstructor:
+ // Function -> Receiver.
+ CheckValueInputIs(node, 0, Type::Function());
+ CheckTypeIs(node, Type::Receiver());
Benedikt Meurer 2016/12/12 18:18:19 Same: Use Type::Callable here.
+ break;
case IrOpcode::kJSLoadContext:
// Type can be anything.
« src/compiler/typer.cc ('K') | « src/compiler/typer.cc ('k') | src/interpreter/bytecode-array-builder.h » ('j') | src/interpreter/interpreter.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698