mirrors/nodejs
mirrors/nodejs
0
0
Fork 0
mirror of https://github.com/nodejs/node.git synced 2024-11-27 22:16:50 +01:00
Code Issues Releases Activity

deps: V8: cherry-pick 7ddb8399f9f1

Browse Source 
Original commit message:

    [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 <ishell@chromium.org>
    Commit-Queue: Jakob Kummerow <jkummerow@chromium.org>
    Reviewed-by: Jakob Kummerow <jkummerow@chromium.org>
    Cr-Commit-Position: refs/heads/main@{#83406}

Refs: 7ddb8399f9
PR-URL: https://github.com/nodejs/node/pull/44741
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Reviewed-By: Jiawen Geng <technicalcute@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
This commit is contained in:
Michaël Zasso 2022-09-25 17:35:13 +02:00
parent 26c651c34e
commit 88ed027d57
No known key found for this signature in database
GPG Key ID: 770F7A9A5AE15600
2 changed files with 11 additions and 11 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
common.gypi
View File

@ -36,7 +36,7 @@
# Reset this number to 0 on major V8 upgrades.
# Increment by one for each non-official patch applied to deps/v8.
'v8_embedder_string': '-node.9',
'v8_embedder_string': '-node.10',
##### V8 defaults for Node.js #####

20
deps/v8/test/cctest/test-assembler-arm64.cc vendored
View File

@ -12179,35 +12179,35 @@ TEST(peek_poke_unaligned) {
// x0-x6 should be unchanged.
// w10-w12 should contain the lower words of x0-x2.
__ Poke(x0, 1);
Clobber(&masm, {x0});
Clobber(&masm, RegList{x0});
__ Peek(x0, 1);
__ Poke(x1, 2);
Clobber(&masm, {x1});
Clobber(&masm, RegList{x1});
__ Peek(x1, 2);
__ Poke(x2, 3);
Clobber(&masm, {x2});
Clobber(&masm, RegList{x2});
__ Peek(x2, 3);
__ Poke(x3, 4);
Clobber(&masm, {x3});
Clobber(&masm, RegList{x3});
__ Peek(x3, 4);
__ Poke(x4, 5);
Clobber(&masm, {x4});
Clobber(&masm, RegList{x4});
__ Peek(x4, 5);
__ Poke(x5, 6);
Clobber(&masm, {x5});
Clobber(&masm, RegList{x5});
__ Peek(x5, 6);
__ Poke(x6, 7);
Clobber(&masm, {x6});
Clobber(&masm, RegList{x6});
__ Peek(x6, 7);
__ Poke(w0, 1);
Clobber(&masm, {w10});
Clobber(&masm, RegList{w10});
__ Peek(w10, 1);
__ Poke(w1, 2);
Clobber(&masm, {w11});
Clobber(&masm, RegList{w11});
__ Peek(w11, 2);
__ Poke(w2, 3);
Clobber(&masm, {w12});
Clobber(&masm, RegList{w12});
__ Peek(w12, 3);
__ Drop(4);