deps: update V8 to 13.3.415.20

[targos]

Notable changes:

• Atomics.pause
• WebAssembly Memory64

@nodejs/v8-update @nodejs/tsc

Supersedes #56842

[nodejs-github-bot]

Review requested:

• @nodejs/gyp
• @nodejs/security-wg
• @nodejs/v8-update

[nodejs-github-bot]

CI: https://ci.nodejs.org/job/node-test-pull-request/65055/

[targos]

@nodejs/platform-windows There are new compiler errors (including with clang-cl this time):
https://ci.nodejs.org/job/node-compile-windows/60341/
https://ci.nodejs.org/job/node-compile-windows-debug/26107/nodes=win-vs2022/

[targos]

There are some crashes in debug mode: https://ci.nodejs.org/job/node-test-commit-arm-debug/17147/

/cc @joyeecheung, seems related to cppgc

[joyeecheung]

The crash looks similar to #56327 - we probably have been forgetting to put the locker somewhere and it's somehow exposed by recent V8 changes. Looking..

[targos]

Ping @nodejs/platform-windows

[joyeecheung]

I cannot reproduce test-child-process-exec-maxbuf locally though #57031 fixes test-heapsnapshot-near-heap-limit-bounded locally for me. From the stack trace it seems they are caused by something similar so it may fix the other one as well.

[joyeecheung]

Pushed the commit from #57031

[joyeecheung]

I don't have a Windows machine at hand at the hour, but I think this should fix it for the windows-clang build from a look at the error

diff --git a/deps/v8/src/wasm/wasm-module-sourcemap.cc b/deps/v8/src/wasm/wasm-module-sourcemap.cc
index f95107faba..c65389839e 100644
--- a/deps/v8/src/wasm/wasm-module-sourcemap.cc
+++ b/deps/v8/src/wasm/wasm-module-sourcemap.cc
@@ -69,7 +69,7 @@ WasmModuleSourceMap::WasmModuleSourceMap(v8::Isolate* v8_isolate,
     size_t file_name_sz = file_name->Utf8LengthV2(v8_isolate) + 1;
     std::unique_ptr<char[]> file_name_buf(new char[file_name_sz]);
     file_name->WriteUtf8V2(v8_isolate, file_name_buf.get(), file_name_sz,
-                           String::WriteFlags::kNullTerminate);
+                           v8::String::WriteFlags::kNullTerminate);
     filenames.emplace_back(file_name_buf.get());
   }

@@ -84,7 +84,7 @@ WasmModuleSourceMap::WasmModuleSourceMap(v8::Isolate* v8_isolate,
   size_t mappings_sz = mappings->Utf8LengthV2(v8_isolate) + 1;
   std::unique_ptr<char[]> mappings_buf(new char[mappings_sz]);
   mappings->WriteUtf8V2(v8_isolate, mappings_buf.get(), mappings_sz,
-                        String::WriteFlags::kNullTerminate);
+                        v8::String::WriteFlags::kNullTerminate);

   valid_ = DecodeMapping(mappings_buf.get());
 }

The MSVC one looks different though.