Skip to content

deps: update V8 to 9.0 #37587

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
targos wants to merge 15 commits into from
Closed

deps: update V8 to 9.0 #37587

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
15 commits
Select commit Hold shift + click to select a range
42b4168
deps: update V8 to 9.0.257.11
targos Mar 12, 2021
5907176
build: reset embedder string to "-node.0"
targos Mar 12, 2021
7669aa9
src: update NODE_MODULE_VERSION to 93
targos Mar 3, 2021
9b2d050
deps: V8: un-cherry-pick bd019bd
refack Mar 27, 2019
2ed3e1b
deps: V8: patch register-arm64.h
refack May 22, 2019
7409ec0
deps: V8: forward declaration of `Rtl*FunctionTable`
refack May 22, 2019
b6b64a8
deps: make v8.h compatible with VS2015
joaocgreis Nov 1, 2019
1430740
deps: fix V8 build issue with inline methods
gengjiawen Oct 14, 2020
f89e610
deps: silence irrelevant V8 warnings
targos Jan 30, 2021
1fe5117
deps: V8: cherry-pick 1648e050cade
targos Mar 12, 2021
0bf6e1e
src: use non-deprecated V8 module APIs
targos Jan 30, 2021
2ab93a5
lib: load v8_prof_processor dependencies as ESM
targos Jan 30, 2021
62710da
tools: update V8 gypfiles for 9.0
targos Jan 29, 2021
a47f565
src: remove V8_FT_ADAPTOR for V8 update
cjihrig Feb 26, 2021
430c7ee
src: use non-deprecated GetCreationContext from V8
targos Feb 28, 2021
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
deps: V8: cherry-pick 1648e050cade 
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
  • Loading branch information
@targos
targos committed Mar 12, 2021
commit 1fe511745e66c383844c891144a445186f680ff2
2 changes: 1 addition & 1 deletion common.gypi
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -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.6',
'v8_embedder_string': '-node.7',

##### V8 defaults for Node.js #####

Expand Down
11 changes: 11 additions & 0 deletions deps/v8/src/torque/torque-parser.cc
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1834,7 +1834,18 @@ base::Optional<ParseResult> MakeNumberLiteralExpression(
// Meanwhile, we type it as constexpr float64 when out of int32 range.
double value = 0;
try {
#if defined(V8_OS_SOLARIS)
// stod() on Solaris does not currently support hex strings. Use strtol()
// specifically for hex literals until stod() support is available.
if (number.find("0x") == std::string::npos &&
number.find("0X") == std::string::npos) {
value = std::stod(number);
} else {
value = static_cast<double>(strtol(number.c_str(), nullptr, 0));
}
#else
value = std::stod(number);
#endif // !defined(V8_OS_SOLARIS)
} catch (const std::out_of_range&) {
Error("double literal out-of-range").Throw();
}
Expand Down