Skip to content

deps: enable new JS features the V8 way #58230

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 5 commits into from
Closed

deps: enable new JS features the V8 way #58230

Changes from 1 commit
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
e3c1960
deps: patch V8 to 13.6.233.10
targos May 8, 2025
e09a299
deps: V8: cherry-pick 044b9b6f589d
rmahdav May 1, 2025
44f091a
deps: V8: cherry-pick 4f38995c8295
syg May 5, 2025
9c2d5d0
deps: V8: backport 1d3362c55396
syg May 6, 2025
fa6d943
src: remove unneeded explicit V8 flags
targos May 8, 2025
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
src: remove unneeded explicit V8 flags
  • Loading branch information
@targos
targos committed May 8, 2025
commit fa6d943642b5b3ad98bf94d80f84991e8674417b
5 changes: 0 additions & 5 deletions src/node.cc
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -845,11 +845,6 @@ static ExitCode InitializeNodeWithArgsInternal(
// is security relevant, for Node it's less important.
V8::SetFlagsFromString("--no-freeze-flags-after-init");

// These features are completed and enabled by default in Chrome, but not
// in V8.
V8::SetFlagsFromString("--js-explicit-resource-management");
V8::SetFlagsFromString("--js-float16array");

#if defined(NODE_V8_OPTIONS)
// Should come before the call to V8::SetFlagsFromCommandLine()
// so the user can disable a flag --foo at run-time by passing
Expand Down
Loading