Skip to content

Commit 7ddb839

Browse files
Pierrick BouvierV8 LUCI CQ
authored andcommitted
[msvc] fix ambiguous call to function in assembler test
MSVC is confused by initializer list and default parameter, and reports an ambiguous call. test/cctest/test-assembler-arm64.cc(12208): error C2668: 'v8::internal::Clobber': ambiguous call to overloaded function test-utils-arm64.h(251): note: could be 'void v8::internal::Clobber(v8::internal::MacroAssembler *,v8::internal::CPURegList)' test-utils-arm64.h(241): note: or 'void v8::internal::Clobber(v8::internal::MacroAssembler *,v8::internal::RegList,const uint64_t)' Solution is to construct with explicit type. Bug: v8:13312 Change-Id: I66f5ba48bcdf6eb30035beaf7214a3d26fc9f18b Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3913034 Reviewed-by: Igor Sheludko <[email protected]> Commit-Queue: Jakob Kummerow <[email protected]> Reviewed-by: Jakob Kummerow <[email protected]> Cr-Commit-Position: refs/heads/main@{#83406}
1 parent 758823a commit 7ddb839

1 file changed

Lines changed: 10 additions & 10 deletions

File tree

test/cctest/test-assembler-arm64.cc

Lines changed: 10 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -12180,35 +12180,35 @@ TEST(peek_poke_unaligned) {
1218012180
// x0-x6 should be unchanged.
1218112181
// w10-w12 should contain the lower words of x0-x2.
1218212182
__ Poke(x0, 1);
12183-
Clobber(&masm, {x0});
12183+
Clobber(&masm, RegList{x0});
1218412184
__ Peek(x0, 1);
1218512185
__ Poke(x1, 2);
12186-
Clobber(&masm, {x1});
12186+
Clobber(&masm, RegList{x1});
1218712187
__ Peek(x1, 2);
1218812188
__ Poke(x2, 3);
12189-
Clobber(&masm, {x2});
12189+
Clobber(&masm, RegList{x2});
1219012190
__ Peek(x2, 3);
1219112191
__ Poke(x3, 4);
12192-
Clobber(&masm, {x3});
12192+
Clobber(&masm, RegList{x3});
1219312193
__ Peek(x3, 4);
1219412194
__ Poke(x4, 5);
12195-
Clobber(&masm, {x4});
12195+
Clobber(&masm, RegList{x4});
1219612196
__ Peek(x4, 5);
1219712197
__ Poke(x5, 6);
12198-
Clobber(&masm, {x5});
12198+
Clobber(&masm, RegList{x5});
1219912199
__ Peek(x5, 6);
1220012200
__ Poke(x6, 7);
12201-
Clobber(&masm, {x6});
12201+
Clobber(&masm, RegList{x6});
1220212202
__ Peek(x6, 7);
1220312203

1220412204
__ Poke(w0, 1);
12205-
Clobber(&masm, {w10});
12205+
Clobber(&masm, RegList{w10});
1220612206
__ Peek(w10, 1);
1220712207
__ Poke(w1, 2);
12208-
Clobber(&masm, {w11});
12208+
Clobber(&masm, RegList{w11});
1220912209
__ Peek(w11, 2);
1221012210
__ Poke(w2, 3);
12211-
Clobber(&masm, {w12});
12211+
Clobber(&masm, RegList{w12});
1221212212
__ Peek(w12, 3);
1221312213

1221412214
__ Drop(4);

0 commit comments

Comments
 (0)