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

Issues Search
    My Issues | Starred     Open | Closed | All

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

Issue 2038323002: [interpreter] Filter expression positions at source. (Closed) Base URL: https://chromium.googlesource.com/v8/v8.git@oth-0060-source-position-testing
Patch Set: Fix expression positions on StackCheck that this CL broke. Created 4 years, 6 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
« src/interpreter/bytecode-array-builder.h ('K') | « test/cctest/interpreter/bytecode_expectations/DeadCodeRemoval.golden ('k') | test/cctest/interpreter/bytecode_expectations/ForIn.golden » ('j') | test/cctest/interpreter/test-source-positions.cc » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: test/cctest/interpreter/bytecode_expectations/DoExpression.golden
diff --git a/test/cctest/interpreter/bytecode_expectations/DoExpression.golden b/test/cctest/interpreter/bytecode_expectations/DoExpression.golden
index 49221ea44a561e93ff42758abe9bf6576a583bbe..dd8a085fc47c66866ddac24a60c9597b6c706f8b 100644
--- a/test/cctest/interpreter/bytecode_expectations/DoExpression.golden
+++ b/test/cctest/interpreter/bytecode_expectations/DoExpression.golden
@@ -14,11 +14,10 @@ snippet: "
"
frame size: 2
parameter count: 1
-bytecode array length: 8
+bytecode array length: 7
bytecodes: [
/* 30 E> */ B(StackCheck),
- /* 42 S> */ B(Nop),
- B(Mov), R(0), R(1),
+ /* 42 S> */ B(Mov), R(0), R(1),
/* 50 S> */ B(Ldar), R(1),
/* 60 S> */ B(Return),
]
@@ -33,14 +32,13 @@ snippet: "
"
frame size: 3
parameter count: 1
-bytecode array length: 14
+bytecode array length: 13
bytecodes: [
/* 30 E> */ B(StackCheck),
/* 55 S> */ B(LdaSmi), U8(100),
B(Star), R(1),
/* 42 S> */ B(LdrUndefined), R(0),
B(Ldar), R(0),
- /* 42 E> */ B(Nop),
B(Star), R(2),
/* 63 S> */ B(Nop),
/* 73 S> */ B(Return),
@@ -56,21 +54,20 @@ snippet: "
"
frame size: 2
parameter count: 1
-bytecode array length: 25
+bytecode array length: 24
bytecodes: [
/* 30 E> */ B(StackCheck),
/* 34 E> */ B(StackCheck),
/* 56 S> */ B(LdaSmi), U8(10),
B(Star), R(1),
/* 69 S> */ B(Inc),
- /* 71 E> */ B(Star), R(1),
+ B(Star), R(1),
B(Star), R(0),
- /* 74 S> */ B(Jump), U8(12),
- /* 64 E> */ B(Nop),
+ /* 74 S> */ B(Jump), U8(11),
B(Mov), R(0), R(1),
/* 84 S> */ B(LdaSmi), U8(20),
- /* 86 E> */ B(Star), R(1),
- B(Jump), U8(-20),
+ B(Star), R(1),
+ B(Jump), U8(-19),
B(LdaUndefined),
/* 94 S> */ B(Return),
]
« src/interpreter/bytecode-array-builder.h ('K') | « test/cctest/interpreter/bytecode_expectations/DeadCodeRemoval.golden ('k') | test/cctest/interpreter/bytecode_expectations/ForIn.golden » ('j') | test/cctest/interpreter/test-source-positions.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698