deps: update V8 to 9.0 #37587

targos · 2021-03-03T17:00:20Z

ETA: Tue, Apr 13, 2021

targos · 2021-03-03T17:01:01Z

CI running at https://ci.nodejs.org/job/node-test-commit/44792/
V8 CI: https://ci.nodejs.org/job/node-test-commit-v8-linux/3819/

aduh95 · 2021-03-04T10:27:39Z

Should we keep f26c461 and dd99628 considering we now require XCode ≥10? Ditto for 22be529 as we require VS ≥2019.

node/BUILDING.md

Lines 158 to 166 in 7a4c2c8

    
           ### Supported toolchains 
        
           Depending on the host platform, the selection of toolchains may vary. 
        
           | Operating System | Compiler Versions                                              | 
        
           | ---------------- | -------------------------------------------------------------- | 
        
           | Linux            | GCC >= 6.3                                                     | 
        
           | Windows          | Visual Studio >= 2019 with the Windows 10 SDK on a 64-bit host | 
        
           | macOS            | Xcode >= 10 (Apple LLVM >= 10)                                 |

targos · 2021-03-04T10:56:27Z

I'll try to remove the XCode patches.
We still support Visual Studio 2015 for building addons. See #37472

nodejs-github-bot · 2021-03-04T11:20:26Z

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

targos · 2021-03-04T12:23:49Z

CITGM: https://ci.nodejs.org/view/Node.js-citgm/job/citgm-smoker/2642/

Major V8 updates are usually API/ABI incompatible with previous versions. This commit adapts NODE_MODULE_VERSION for V8 9.0. Refs: https://github.com/nodejs/CTC/blob/master/meetings/2016-09-28.md

Original commit message: [testrunner] delete ancient junit compatible format support Testrunner has ancient support for JUnit compatible XML output. This CL removes this old feature. R=mstarzinger@chromium.org,jgruber@chromium.org,jkummerow@chromium.org CC=machenbach@chromium.org Bug: v8:8728 Change-Id: I7e1beb011dbaec3aa1a27398a5c52abdd778eaf0 Reviewed-on: https://chromium-review.googlesource.com/c/1430065 Reviewed-by: Jakob Gruber <jgruber@chromium.org> Reviewed-by: Michael Starzinger <mstarzinger@chromium.org> Commit-Queue: Tamer Tas <tmrts@chromium.org> Cr-Commit-Position: refs/heads/master@{#59045} Refs: v8/v8@bd019bd PR-URL: #32116 Reviewed-By: Michaël Zasso <targos@protonmail.com> Reviewed-By: Jiawen Geng <technicalcute@gmail.com> Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>

Fixes a compilation issue on some platforms PR-URL: #32116 Reviewed-By: Michaël Zasso <targos@protonmail.com> Reviewed-By: Jiawen Geng <technicalcute@gmail.com> Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>

This should be semver-patch since actual invocation is version conditional. PR-URL: #32116 Reviewed-By: Michaël Zasso <targos@protonmail.com> Reviewed-By: Jiawen Geng <technicalcute@gmail.com> Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>

There is a bug in the most recent version of VS2015 that affects v8.h and therefore prevents compilation of addons. Refs: https://stackoverflow.com/q/38378693 PR-URL: #32116 Reviewed-By: Michaël Zasso <targos@protonmail.com> Reviewed-By: Jiawen Geng <technicalcute@gmail.com> Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>

PR-URL: #35415 Reviewed-By: Rich Trott <rtrott@gmail.com> Reviewed-By: Jiawen Geng <technicalcute@gmail.com> Reviewed-By: Daniel Bevenius <daniel.bevenius@gmail.com> Reviewed-By: Matteo Collina <matteo.collina@gmail.com> Reviewed-By: Myles Borins <myles.borins@gmail.com>

Original commit message: torque: workaround stod() limitations on Solaris std::stod() on Solaris does not currently handle hex strings. This commit provides a workaround based on strtol() until proper stod() support is available. This was encountered while updating Node.js to V8 8.8. For more details see the following comment: #36139 (comment) Change-Id: I16ed80a817f6d9105e7153b10824b1fee8520432 Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2692746 Reviewed-by: Michael Stanton <mvstanton@chromium.org> Commit-Queue: Michael Stanton <mvstanton@chromium.org> Cr-Commit-Position: refs/heads/master@{#73255} Refs: v8/v8@1648e05

The script versions are no longer available.

V8 has removed argument adapter frames. This commit removes them from the Node.js codebase as well.

Fixes: nodejs/node-v8#193

nodejs-github-bot · 2021-03-12T08:18:26Z

CI: https://ci.nodejs.org/job/node-test-pull-request/36648/
V8: https://ci.nodejs.org/job/node-test-commit-v8-linux/3834/

PR-URL: nodejs#37587 Reviewed-By: Jiawen Geng <technicalcute@gmail.com> Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Myles Borins <myles.borins@gmail.com>