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

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

Issue 1997653002: [interpreter] Bytecode register optimizer. (Closed) Base URL: https://chromium.googlesource.com/v8/v8.git@master
Patch Set: Additional comment. Created 4 years, 7 months 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/ObjectLiterals.golden
diff --git a/test/cctest/interpreter/bytecode_expectations/ObjectLiterals.golden b/test/cctest/interpreter/bytecode_expectations/ObjectLiterals.golden
index b84f7b33d7a6d01520c1eddca61befc50bfcdebf..47a70bd62ce0f90b06f8304fbce3f40d4b4bcfed 100644
--- a/test/cctest/interpreter/bytecode_expectations/ObjectLiterals.golden
+++ b/test/cctest/interpreter/bytecode_expectations/ObjectLiterals.golden
@@ -57,8 +57,8 @@ bytecodes: [
/* 42 S> */ B(LdaSmi), U8(1),
B(Star), R(0),
/* 45 S> */ B(CreateObjectLiteral), U8(0), U8(0), U8(1),
- B(Star), R(1),
- /* 75 E> */ B(Ldar), R(0),
+ /* 75 E> */ B(Star), R(1),
+ B(Ldar), R(0),
B(StaNamedPropertySloppy), R(1), U8(1), U8(1),
B(Ldar), R(1),
/* 80 S> */ B(Return),
@@ -74,19 +74,17 @@ handlers: [
snippet: "
var a = 1; return { val: a, val: a + 1 };
"
-frame size: 3
+frame size: 2
parameter count: 1
-bytecode array length: 26
+bytecode array length: 22
bytecodes: [
/* 30 E> */ B(StackCheck),
/* 42 S> */ B(LdaSmi), U8(1),
B(Star), R(0),
/* 45 S> */ B(CreateObjectLiteral), U8(0), U8(0), U8(1),
- B(Star), R(1),
- /* 59 E> */ B(Ldar), R(0),
- /* 67 E> */ B(Star), R(2),
- B(LdaSmi), U8(1),
- B(Add), R(2),
+ /* 59 E> */ B(Star), R(1),
+ /* 67 E> */ B(LdaSmi), U8(1),
+ B(Add), R(0),
B(StaNamedPropertySloppy), R(1), U8(1), U8(1),
B(Ldar), R(1),
/* 76 S> */ B(Return),
@@ -157,7 +155,6 @@ bytecodes: [
/* 30 E> */ B(StackCheck),
/* 34 S> */ B(CreateObjectLiteral), U8(0), U8(0), U8(1),
B(Star), R(0),
- B(Mov), R(0), R(1),
B(LdaConstant), U8(1),
B(Star), R(2),
B(CreateClosure), U8(2), U8(0),
@@ -166,6 +163,7 @@ bytecodes: [
B(Star), R(4),
B(LdaZero),
B(Star), R(5),
+ B(Mov), R(0), R(1),
B(CallRuntime), U16(Runtime::kDefineAccessorPropertyUnchecked), R(1), U8(5),
B(Ldar), R(0),
/* 68 S> */ B(Return),
@@ -189,7 +187,6 @@ bytecodes: [
/* 30 E> */ B(StackCheck),
/* 34 S> */ B(CreateObjectLiteral), U8(0), U8(0), U8(1),
B(Star), R(0),
- B(Mov), R(0), R(1),
B(LdaConstant), U8(1),
B(Star), R(2),
B(CreateClosure), U8(2), U8(0),
@@ -198,6 +195,7 @@ bytecodes: [
B(Star), R(4),
B(LdaZero),
B(Star), R(5),
+ B(Mov), R(0), R(1),
B(CallRuntime), U16(Runtime::kDefineAccessorPropertyUnchecked), R(1), U8(5),
B(Ldar), R(0),
/* 102 S> */ B(Return),
@@ -222,7 +220,6 @@ bytecodes: [
/* 30 E> */ B(StackCheck),
/* 34 S> */ B(CreateObjectLiteral), U8(0), U8(0), U8(1),
B(Star), R(0),
- B(Mov), R(0), R(1),
B(LdaConstant), U8(1),
B(Star), R(2),
B(LdaNull),
@@ -231,6 +228,7 @@ bytecodes: [
B(Star), R(4),
B(LdaZero),
B(Star), R(5),
+ B(Mov), R(0), R(1),
B(CallRuntime), U16(Runtime::kDefineAccessorPropertyUnchecked), R(1), U8(5),
B(Ldar), R(0),
/* 74 S> */ B(Return),
@@ -249,20 +247,19 @@ snippet: "
"
frame size: 6
parameter count: 1
-bytecode array length: 33
+bytecode array length: 32
bytecodes: [
/* 30 E> */ B(StackCheck),
/* 42 S> */ B(LdaSmi), U8(1),
B(Star), R(0),
/* 45 S> */ B(CreateObjectLiteral), U8(0), U8(0), U8(1),
B(Star), R(1),
- B(Mov), R(1), R(2),
B(LdaSmi), U8(1),
- B(Star), R(3),
- /* 57 E> */ B(Ldar), R(0),
- B(Star), R(4),
+ /* 57 E> */ B(Star), R(3),
B(LdaZero),
B(Star), R(5),
+ B(Mov), R(1), R(2),
+ B(Mov), R(0), R(4),
B(CallRuntime), U16(Runtime::kSetProperty), R(2), U8(4),
B(Ldar), R(1),
/* 62 S> */ B(Return),
@@ -284,9 +281,9 @@ bytecodes: [
/* 30 E> */ B(StackCheck),
/* 34 S> */ B(CreateObjectLiteral), U8(0), U8(0), U8(35),
B(Star), R(0),
- B(Mov), R(0), R(1),
B(LdaNull),
B(Star), R(2),
+ B(Mov), R(0), R(1),
B(CallRuntime), U16(Runtime::kInternalSetPrototype), R(1), U8(2),
B(Ldar), R(0),
/* 62 S> */ B(Return),
@@ -309,9 +306,8 @@ bytecodes: [
/* 42 S> */ B(LdaConstant), U8(0),
B(Star), R(0),
/* 50 S> */ B(CreateObjectLiteral), U8(1), U8(0), U8(35),
- B(Star), R(1),
- B(Mov), R(1), R(2),
- /* 60 E> */ B(Ldar), R(0),
+ /* 60 E> */ B(Star), R(1),
+ B(Ldar), R(0),
B(ToName),
B(Star), R(3),
B(LdaSmi), U8(1),
@@ -320,6 +316,7 @@ bytecodes: [
B(Star), R(5),
B(LdaZero),
B(Star), R(6),
+ B(Mov), R(1), R(2),
B(CallRuntime), U16(Runtime::kDefineDataPropertyInLiteral), R(2), U8(5),
B(Ldar), R(1),
/* 69 S> */ B(Return),
@@ -337,18 +334,16 @@ snippet: "
"
frame size: 7
parameter count: 1
-bytecode array length: 43
+bytecode array length: 41
bytecodes: [
/* 30 E> */ B(StackCheck),
/* 42 S> */ B(LdaConstant), U8(0),
B(Star), R(0),
/* 50 S> */ B(CreateObjectLiteral), U8(1), U8(0), U8(1),
- B(Star), R(1),
- /* 64 E> */ B(Ldar), R(0),
+ /* 64 E> */ B(Star), R(1),
+ B(Ldar), R(0),
B(StaNamedPropertySloppy), R(1), U8(2), U8(1),
- B(Mov), R(1), R(2),
- /* 68 E> */ B(Ldar), R(0),
- B(ToName),
+ /* 68 E> */ B(ToName),
B(Star), R(3),
B(LdaSmi), U8(1),
B(Star), R(4),
@@ -356,6 +351,7 @@ bytecodes: [
B(Star), R(5),
B(LdaZero),
B(Star), R(6),
+ B(Mov), R(1), R(2),
B(CallRuntime), U16(Runtime::kDefineDataPropertyInLiteral), R(2), U8(5),
B(Ldar), R(1),
/* 77 S> */ B(Return),
@@ -374,15 +370,14 @@ snippet: "
"
frame size: 7
parameter count: 1
-bytecode array length: 53
+bytecode array length: 54
bytecodes: [
/* 30 E> */ B(StackCheck),
/* 42 S> */ B(LdaConstant), U8(0),
B(Star), R(0),
/* 50 S> */ B(CreateObjectLiteral), U8(1), U8(1), U8(35),
- B(Star), R(1),
- B(Mov), R(1), R(2),
- /* 60 E> */ B(Ldar), R(0),
+ /* 60 E> */ B(Star), R(1),
+ B(Ldar), R(0),
B(ToName),
B(Star), R(3),
B(LdaSmi), U8(1),
@@ -391,11 +386,12 @@ bytecodes: [
B(Star), R(5),
B(LdaZero),
B(Star), R(6),
- B(CallRuntime), U16(Runtime::kDefineDataPropertyInLiteral), R(2), U8(5),
B(Mov), R(1), R(2),
+ B(CallRuntime), U16(Runtime::kDefineDataPropertyInLiteral), R(2), U8(5),
B(CreateObjectLiteral), U8(1), U8(0), U8(35),
B(Star), R(4),
- B(Star), R(3),
+ B(Mov), R(1), R(2),
+ B(Mov), R(4), R(3),
B(CallRuntime), U16(Runtime::kInternalSetPrototype), R(2), U8(2),
B(Ldar), R(1),
/* 84 S> */ B(Return),
@@ -419,9 +415,8 @@ bytecodes: [
/* 42 S> */ B(LdaConstant), U8(0),
B(Star), R(0),
/* 50 S> */ B(CreateObjectLiteral), U8(1), U8(0), U8(35),
- B(Star), R(1),
- B(Mov), R(1), R(2),
- /* 60 E> */ B(Ldar), R(0),
+ /* 60 E> */ B(Star), R(1),
+ B(Ldar), R(0),
B(ToName),
B(Star), R(3),
B(LdaConstant), U8(2),
@@ -430,22 +425,23 @@ bytecodes: [
B(Star), R(5),
B(LdaZero),
B(Star), R(6),
- B(CallRuntime), U16(Runtime::kDefineDataPropertyInLiteral), R(2), U8(5),
B(Mov), R(1), R(2),
+ B(CallRuntime), U16(Runtime::kDefineDataPropertyInLiteral), R(2), U8(5),
B(LdaConstant), U8(3),
B(Star), R(3),
B(CreateClosure), U8(4), U8(0),
B(Star), R(4),
B(LdaZero),
B(Star), R(5),
- B(CallRuntime), U16(Runtime::kDefineGetterPropertyUnchecked), R(2), U8(4),
B(Mov), R(1), R(2),
+ B(CallRuntime), U16(Runtime::kDefineGetterPropertyUnchecked), R(2), U8(4),
B(LdaConstant), U8(3),
B(Star), R(3),
B(CreateClosure), U8(5), U8(0),
B(Star), R(4),
B(LdaZero),
B(Star), R(5),
+ B(Mov), R(1), R(2),
B(CallRuntime), U16(Runtime::kDefineSetterPropertyUnchecked), R(2), U8(4),
B(Ldar), R(1),
/* 99 S> */ B(Return),

Powered by Google App Engine
This is Rietveld 408576698