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

Side by Side Diff: test/unittests/interpreter/bytecode-decoder-unittest.cc

Issue 2382273002: [Interpreter]: Add kRegList operand type for register list operands. (Closed)
Patch Set: Created 4 years, 2 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 unified diff | Download patch
OLDNEW
1 // Copyright 2015 the V8 project authors. All rights reserved. 1 // Copyright 2015 the V8 project authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include <vector> 5 #include <vector>
6 6
7 #include "src/v8.h" 7 #include "src/v8.h"
8 8
9 #include "src/interpreter/bytecode-decoder.h" 9 #include "src/interpreter/bytecode-decoder.h"
10 #include "test/unittests/interpreter/bytecode-utils.h" 10 #include "test/unittests/interpreter/bytecode-utils.h"
(...skipping 19 matching lines...) Expand all
30 0, 30 0,
31 "LdaSmi.ExtraWide [100000]"}, 31 "LdaSmi.ExtraWide [100000]"},
32 {{B(LdaSmi), U8(-1)}, 2, 0, " LdaSmi [-1]"}, 32 {{B(LdaSmi), U8(-1)}, 2, 0, " LdaSmi [-1]"},
33 {{B(Wide), B(LdaSmi), U16(-1000)}, 4, 0, " LdaSmi.Wide [-1000]"}, 33 {{B(Wide), B(LdaSmi), U16(-1000)}, 4, 0, " LdaSmi.Wide [-1000]"},
34 {{B(ExtraWide), B(LdaSmi), U32(-100000)}, 34 {{B(ExtraWide), B(LdaSmi), U32(-100000)},
35 6, 35 6,
36 0, 36 0,
37 "LdaSmi.ExtraWide [-100000]"}, 37 "LdaSmi.ExtraWide [-100000]"},
38 {{B(Star), R8(5)}, 2, 0, " Star r5"}, 38 {{B(Star), R8(5)}, 2, 0, " Star r5"},
39 {{B(Wide), B(Star), R16(136)}, 4, 0, " Star.Wide r136"}, 39 {{B(Wide), B(Star), R16(136)}, 4, 0, " Star.Wide r136"},
40 {{B(Wide), B(Call), R16(134), R16(135), U16(2), U16(177)}, 40 {{B(Wide), B(Call), R16(134), R16(135), U16(10), U16(177)},
41 10, 41 10,
42 0, 42 0,
43 "Call.Wide r134, r135, #2, [177]"}, 43 "Call.Wide r134, r135-r144, [177]"},
44 {{B(ForInPrepare), R8(10), R8(11)},
45 3,
46 0,
47 " ForInPrepare r10, r11-r13"},
mythria 2016/10/03 10:29:34 May be one more test case with Reg count of 0 for
rmcilroy 2016/10/05 15:48:45 Good point - done.
44 {{B(Ldar), 48 {{B(Ldar),
45 static_cast<uint8_t>(Register::FromParameterIndex(2, 3).ToOperand())}, 49 static_cast<uint8_t>(Register::FromParameterIndex(2, 3).ToOperand())},
46 2, 50 2,
47 3, 51 3,
48 " Ldar a1"}, 52 " Ldar a1"},
49 {{B(Wide), B(CreateObjectLiteral), U16(513), U16(1027), U8(165), 53 {{B(Wide), B(CreateObjectLiteral), U16(513), U16(1027), U8(165),
50 R16(137)}, 54 R16(137)},
51 9, 55 9,
52 0, 56 0,
53 "CreateObjectLiteral.Wide [513], [1027], #165, r137"}, 57 "CreateObjectLiteral.Wide [513], [1027], #165, r137"},
(...skipping 24 matching lines...) Expand all
78 cases[i].parameter_count); 82 cases[i].parameter_count);
79 83
80 // Compare. 84 // Compare.
81 CHECK_EQ(actual_ss.str(), expected_ss.str()); 85 CHECK_EQ(actual_ss.str(), expected_ss.str());
82 } 86 }
83 } 87 }
84 88
85 } // namespace interpreter 89 } // namespace interpreter
86 } // namespace internal 90 } // namespace internal
87 } // namespace v8 91 } // namespace v8
OLDNEW
« no previous file with comments | « src/interpreter/bytecodes.h ('k') | test/unittests/interpreter/interpreter-assembler-unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698