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

Unified Diff: test/cctest/interpreter/bytecode_expectations/ClassDeclarations.golden

Issue 2423053002: Install the 'name' property in classes at runtime (Closed)
Patch Set: Move computed property names check to parser and runtime function Created 4 years, 1 month 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
Index: test/cctest/interpreter/bytecode_expectations/ClassDeclarations.golden
diff --git a/test/cctest/interpreter/bytecode_expectations/ClassDeclarations.golden b/test/cctest/interpreter/bytecode_expectations/ClassDeclarations.golden
index f1a15639a885b2ad78dfb1f24106e15ff9fc632d..eeb354e485dab2d5019b7a94f7a463e27e0a9a42 100644
--- a/test/cctest/interpreter/bytecode_expectations/ClassDeclarations.golden
+++ b/test/cctest/interpreter/bytecode_expectations/ClassDeclarations.golden
@@ -14,7 +14,7 @@ snippet: "
"
frame size: 10
parameter count: 1
-bytecode array length: 74
+bytecode array length: 85
bytecodes: [
B(LdaTheHole),
B(Star), R(2),
@@ -42,6 +42,10 @@ bytecodes: [
B(Star), R(9),
B(Mov), R(4), R(5),
B(CallRuntime), U16(Runtime::kDefineDataPropertyInLiteral), R(5), U8(5),
+ B(LdaFalse),
+ B(Star), R(6),
+ B(Mov), R(3), R(5),
+ B(CallRuntime), U16(Runtime::kInstallClassNameAccessor), R(5), U8(2),
B(CallRuntime), U16(Runtime::kToFastProperties), R(3), U8(1),
B(Star), R(0),
B(Star), R(1),
@@ -67,7 +71,7 @@ snippet: "
"
frame size: 10
parameter count: 1
-bytecode array length: 74
+bytecode array length: 85
bytecodes: [
B(LdaTheHole),
B(Star), R(2),
@@ -95,6 +99,10 @@ bytecodes: [
B(Star), R(9),
B(Mov), R(4), R(5),
B(CallRuntime), U16(Runtime::kDefineDataPropertyInLiteral), R(5), U8(5),
+ B(LdaFalse),
+ B(Star), R(6),
+ B(Mov), R(3), R(5),
+ B(CallRuntime), U16(Runtime::kInstallClassNameAccessor), R(5), U8(2),
B(CallRuntime), U16(Runtime::kToFastProperties), R(3), U8(1),
B(Star), R(0),
B(Star), R(1),
@@ -122,7 +130,7 @@ snippet: "
"
frame size: 11
parameter count: 1
-bytecode array length: 128
+bytecode array length: 139
bytecodes: [
B(CreateFunctionContext), U8(2),
B(PushContext), R(3),
@@ -168,6 +176,10 @@ bytecodes: [
B(LdaSmi), U8(1),
B(Star), R(10),
B(CallRuntime), U16(Runtime::kDefineDataPropertyInLiteral), R(6), U8(5),
+ B(LdaTrue),
+ B(Star), R(7),
+ B(Mov), R(4), R(6),
+ B(CallRuntime), U16(Runtime::kInstallClassNameAccessor), R(6), U8(2),
B(CallRuntime), U16(Runtime::kToFastProperties), R(4), U8(1),
B(Star), R(0),
B(Star), R(1),
@@ -194,7 +206,7 @@ snippet: "
"
frame size: 8
parameter count: 1
-bytecode array length: 74
+bytecode array length: 85
bytecodes: [
B(CreateFunctionContext), U8(1),
B(PushContext), R(3),
@@ -216,6 +228,10 @@ bytecodes: [
B(CallRuntime), U16(Runtime::kDefineClass), R(4), U8(4),
B(Star), R(4),
B(LdrNamedProperty), R(4), U8(1), U8(2), R(5),
+ B(LdaFalse),
+ B(Star), R(7),
+ B(Mov), R(4), R(6),
+ B(CallRuntime), U16(Runtime::kInstallClassNameAccessor), R(6), U8(2),
B(CallRuntime), U16(Runtime::kToFastProperties), R(4), U8(1),
B(Star), R(0),
B(Star), R(1),
@@ -236,3 +252,68 @@ constant pool: [
handlers: [
]
+---
+snippet: "
+ (class {})
+ class E { static name () {}}
+"
+frame size: 11
+parameter count: 1
+bytecode array length: 107
+bytecodes: [
+ B(LdaTheHole),
+ B(Star), R(3),
+ /* 30 E> */ B(StackCheck),
+ /* 35 S> */ B(LdaTheHole),
+ B(Star), R(4),
+ B(CreateClosure), U8(0), U8(2),
+ B(Star), R(5),
+ B(LdaSmi), U8(35),
+ B(Star), R(6),
+ B(LdaSmi), U8(43),
+ B(Star), R(7),
+ B(CallRuntime), U16(Runtime::kDefineClass), R(4), U8(4),
+ B(Star), R(4),
+ B(LdrNamedProperty), R(4), U8(1), U8(2), R(5),
+ B(CallRuntime), U16(Runtime::kToFastProperties), R(4), U8(1),
+ B(Star), R(1),
+ B(LdaTheHole),
+ B(Star), R(0),
+ /* 45 S> */ B(LdaTheHole),
+ B(Star), R(4),
+ B(CreateClosure), U8(2), U8(2),
+ B(Star), R(5),
+ B(LdaSmi), U8(45),
+ B(Star), R(6),
+ B(LdaSmi), U8(73),
+ B(Star), R(7),
+ B(CallRuntime), U16(Runtime::kDefineClass), R(4), U8(4),
+ B(Star), R(4),
+ B(LdrNamedProperty), R(4), U8(1), U8(4), R(5),
+ B(LdaConstant), U8(3),
+ B(ToName), R(7),
+ B(CreateClosure), U8(4), U8(2),
+ B(Star), R(8),
+ B(LdaSmi), U8(2),
+ B(Star), R(9),
+ B(LdaZero),
+ B(Star), R(10),
+ B(Mov), R(4), R(6),
+ B(CallRuntime), U16(Runtime::kDefineDataPropertyInLiteral), R(6), U8(5),
+ B(CallRuntime), U16(Runtime::kToFastProperties), R(4), U8(1),
+ B(Star), R(0),
+ B(Star), R(2),
+ B(Star), R(3),
+ B(LdaUndefined),
+ /* 74 S> */ B(Return),
+]
+constant pool: [
+ SHARED_FUNCTION_INFO_TYPE,
+ ONE_BYTE_INTERNALIZED_STRING_TYPE ["prototype"],
+ SHARED_FUNCTION_INFO_TYPE,
+ ONE_BYTE_INTERNALIZED_STRING_TYPE ["name"],
+ SHARED_FUNCTION_INFO_TYPE,
+]
+handlers: [
+]
+

Powered by Google App Engine
This is Rietveld 408576698