Skip to content

[help wanted] deps: update V8 to 13.4 #57114

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.

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
wants to merge 37 commits into from
Closed
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
37 commits
Select commit Hold shift + click to select a range
44cccbe
deps: update V8 to 13.4.114.21
targos Apr 1, 2025
42f6adc
build: reset embedder string to "-node.0"
targos Apr 1, 2025
f3f4d5a
src: update NODE_MODULE_VERSION to 136
targos Feb 26, 2025
54c8717
deps: always define V8_EXPORT_PRIVATE as no-op
targos Sep 21, 2022
46d8aa0
deps: disable V8 concurrent sparkplug compilation
targos Apr 6, 2023
1da6497
deps: patch V8 to avoid duplicated zlib symbol
targos Sep 16, 2023
61b27b4
deps: patch V8 to support compilation with MSVC
StefanStojanovic Apr 21, 2024
17d1f7b
deps: fix FP16 bitcasts.h
StefanStojanovic May 28, 2024
a5c5ae3
deps: always define V8_NODISCARD as no-op
targos Aug 8, 2024
d73d8ab
deps: define V8_PRESERVE_MOST as no-op on Windows
StefanStojanovic Dec 16, 2024
a435680
deps: remove problematic comment from v8-internal
targos Mar 19, 2025
2025345
deps: patch V8 to fix armv7 build
targos Feb 25, 2025
a625a98
deps: patch V8 13.4 to support compilation with MSVC
StefanStojanovic Apr 1, 2025
517d514
deps: silence internal V8 deprecation warning
targos Apr 2, 2025
a546d83
deps: V8: cherry-pick a580b463ac35
joyeecheung Feb 14, 2025
59a87d1
deps: V8: backport 2a23d9fa5217
anonrig Feb 20, 2025
3b25814
deps: V8: backport c24df23717b6
legendecas Mar 12, 2025
950060b
deps: V8: cherry-pick c172ffc5bf54
tunz Mar 6, 2025
58b1ea4
build: remove support for s390 32-bit
richardlau Sep 18, 2024
e47a369
build: enable shared RO heap with ptr compression
targos Sep 21, 2024
a9842d4
tools: update V8 gypfiles for 13.1
targos Sep 25, 2024
97a3cfc
tools: update V8 gypfiles for 13.2
targos Oct 28, 2024
38403e2
build,src,tools: adapt build config for V8 13.3
targos Dec 19, 2024
a5e14b5
tools: update V8 gypfiles for 13.4
targos Jan 20, 2025
fadba44
build: add `/bigobj` to compile V8 on Windows
targos Feb 5, 2025
f50c28c
src: replace uses of FastApiTypedArray
targos Feb 8, 2025
0d0c8ae
Revert "test: disable fast API call count checks"
targos Jan 31, 2025
4bb419b
test: update test-linux-perf-logger
targos Feb 4, 2025
2f3618e
test: adapt assert tests to stack trace changes
targos Dec 20, 2024
f9168fc
test: handle explicit resource management globals
targos Dec 21, 2024
a1abd03
deps: remove deps/simdutf
targos Jan 29, 2025
ab90d04
tools: update license-builder and LICENSE for V8 deps
targos Jan 29, 2025
160062e
src: use `v8::ExternalMemoryAccounter`
targos Feb 15, 2025
6764e4a
src,test: add V8 API to test the hash seed
targos Feb 20, 2025
489cf80
build: pass `-fPIC` to linker as well for shared builds
targos Apr 2, 2025
59feb96
zlib: fix pointer alignment
jhofstee Apr 1, 2025
48b3aee
build: fix V8 TLS config for shared lib builds
targos Apr 3, 2025
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
The table of contents is too big for display.
Diff view
Diff view
  •  
  •  
  •  
The diff you're trying to view is too large. We only load the first 3000 changed files.
4 changes: 2 additions & 2 deletions LICENSE
Original file line number Diff line number Diff line change
Expand Up @@ -1489,7 +1489,7 @@ The externally maintained libraries used by Node.js are:
This code is copyrighted by Sun Microsystems Inc. and released
under a 3-clause BSD license.

- Valgrind client API header, located at src/third_party/valgrind/valgrind.h
- Valgrind client API header, located at third_party/valgrind/valgrind.h
This is released under the BSD license.

- The Wasm C/C++ API headers, located at third_party/wasm-api/wasm.{h,hh}
Expand Down Expand Up @@ -1774,7 +1774,7 @@ The externally maintained libraries used by Node.js are:
limitations under the License.
"""

- simdutf, located at deps/simdutf, is licensed as follows:
- simdutf, located at deps/v8/third_party/simdutf, is licensed as follows:
"""
Copyright 2021 The simdutf authors

Expand Down
16 changes: 1 addition & 15 deletions Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -654,10 +654,6 @@ test-internet: all ## Run internet tests.
test-tick-processor: all ## Run tick processor tests.
$(PYTHON) tools/test.py $(PARALLEL_ARGS) tick-processor

.PHONY: test-hash-seed
test-hash-seed: all ## Verifu that the hash seed used by V8 for hashing is random.
$(NODE) test/pummel/test-hash-seed.js

.PHONY: test-doc
test-doc: doc-only lint-md ## Build, lint, and verify the docs.
@if [ "$(shell $(node_use_openssl_and_icu))" != "true" ]; then \
Expand Down Expand Up @@ -751,8 +747,6 @@ test-v8: v8 ## Run the V8 test suite on deps/v8.
mjsunit cctest debugger inspector message preparser \
$(TAP_V8)
$(call convert_to_junit,$(TAP_V8_JSON))
$(info Testing hash seed)
$(MAKE) test-hash-seed

test-v8-intl: v8 ## Run the v8 test suite, intl tests.
export PATH="$(NO_BIN_OVERRIDE_PATH)" && \
Expand All @@ -768,7 +762,7 @@ test-v8-benchmarks: v8 ## Run the v8 test suite, benchmarks.
$(TAP_V8_BENCHMARKS)
$(call convert_to_junit,$(TAP_V8_BENCHMARKS_JSON))

test-v8-updates: ## Run the v8 test suite, updates.
test-v8-updates: all ## Run the v8 test suite, updates.
$(PYTHON) tools/test.py $(PARALLEL_ARGS) --mode=$(BUILDTYPE_LOWER) v8-updates

test-v8-all: test-v8 test-v8-intl test-v8-benchmarks test-v8-updates ## Run the entire V8 test suite, including intl, benchmarks, and updates.
Expand Down Expand Up @@ -949,9 +943,6 @@ else
ifeq ($(findstring s390x,$(UNAME_M)),s390x)
DESTCPU ?= s390x
else
ifeq ($(findstring s390,$(UNAME_M)),s390)
DESTCPU ?= s390
else
ifeq ($(findstring OS/390,$(shell uname -s)),OS/390)
DESTCPU ?= s390x
else
Expand Down Expand Up @@ -985,7 +976,6 @@ endif
endif
endif
endif
endif
ifeq ($(DESTCPU),x64)
ARCH=x64
else
Expand All @@ -998,9 +988,6 @@ else
ifeq ($(DESTCPU),ppc64)
ARCH=ppc64
else
ifeq ($(DESTCPU),s390)
ARCH=s390
else
ifeq ($(DESTCPU),s390x)
ARCH=s390x
else
Expand All @@ -1018,7 +1005,6 @@ endif
endif
endif
endif
endif

# node and v8 use different arch names (e.g. node 'x86' vs v8 'ia32').
# pass the proper v8 arch name to $V8_ARCH based on user-specified $DESTCPU.
Expand Down
3 changes: 2 additions & 1 deletion common.gypi
Original file line number Diff line number Diff line change
Expand Up @@ -38,7 +38,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.12',
'v8_embedder_string': '-node.15',

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

Expand Down Expand Up @@ -568,6 +568,7 @@
}],
[ 'node_shared=="true"', {
'cflags': [ '-fPIC' ],
'ldflags': [ '-fPIC' ],
}],
],
}],
Expand Down
7 changes: 0 additions & 7 deletions configure.py
Original file line number Diff line number Diff line change
Expand Up @@ -640,12 +640,6 @@
default=None,
help='[Experimental] Enable V8 pointer compression (limits max heap to 4GB and breaks ABI compatibility)')

parser.add_argument('--disable-shared-readonly-heap',
action='store_true',
dest='disable_shared_ro_heap',
default=None,
help='Disable the shared read-only heap feature in V8')

parser.add_argument('--v8-options',
action='store',
dest='v8_options',
Expand Down Expand Up @@ -1720,7 +1714,6 @@ def configure_v8(o, configs):
o['variables']['v8_enable_pointer_compression'] = 1 if options.enable_pointer_compression else 0
o['variables']['v8_enable_sandbox'] = 1 if options.enable_pointer_compression else 0
o['variables']['v8_enable_31bit_smis_on_64bit_arch'] = 1 if options.enable_pointer_compression else 0
o['variables']['v8_enable_shared_ro_heap'] = 0 if options.enable_pointer_compression or options.disable_shared_ro_heap else 1
o['variables']['v8_enable_extensible_ro_snapshot'] = 0
o['variables']['v8_trace_maps'] = 1 if options.trace_maps else 0
o['variables']['node_use_v8_platform'] = b(not options.without_v8_platform)
Expand Down
14 changes: 0 additions & 14 deletions deps/simdutf/BUILD.gn

This file was deleted.

13 changes: 0 additions & 13 deletions deps/simdutf/README.md

This file was deleted.

21 changes: 0 additions & 21 deletions deps/simdutf/simdutf.gyp

This file was deleted.

32 changes: 0 additions & 32 deletions deps/simdutf/unofficial.gni

This file was deleted.

7 changes: 6 additions & 1 deletion deps/v8/.bazelrc
Original file line number Diff line number Diff line change
Expand Up @@ -2,10 +2,14 @@
# Use of this source code is governed by a BSD-style license that can be
# found in the LICENSE file.

# Enable Bzlmod for every Bazel command
common --enable_bzlmod

# Pass CC, CXX and PATH from the environment
build --action_env=CC
build --action_env=CXX
build --action_env=PATH
build --copt=-std=c++20

# Use Clang compiler
build:clang --action_env=BAZEL_COMPILER=clang
Expand All @@ -17,7 +21,8 @@ build:debug --compilation_mode=dbg
build:debug --config=v8_enable_debugging_features
build:debug --//:v8_enable_fast_mksnapshot
build:debug --//:v8_enable_backtrace
build:debug --//:v8_enable_handle_zapping
build:debug --//:v8_enable_local_handle_zapping
build:debug --//:v8_enable_global_handle_zapping

# v8_enable_debugging_features flags
build:v8_enable_debugging_features --//:v8_enable_verify_heap
Expand Down
2 changes: 2 additions & 0 deletions deps/v8/.clang-tidy
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,9 @@
modernize-redundant-void-arg,
modernize-replace-random-shuffle,
modernize-shrink-to-fit,
modernize-type-traits,
# modernize-use-auto,
modernize-use-constraints,
modernize-use-bool-literals,
modernize-use-equals-default,
# modernize-use-equals-delete,
Expand Down
12 changes: 10 additions & 2 deletions deps/v8/.gitignore
Original file line number Diff line number Diff line change
Expand Up @@ -82,14 +82,21 @@
/third_party/googletest/src/googletest/include/gtest/*
!/third_party/googletest/src/googletest/include/gtest/gtest_prod.h
!/third_party/highway
/third_party/highway/src
/third_party/highway/src/*
!/third_party/highway/src/hwy
!/third_party/inspector_protocol
!/third_party/jsoncpp
/third_party/jsoncpp/source
!/third_party/llvm-libc
/third_party/llvm-libc/src
!/third_party/rapidhash-v8
!/third_party/re2
/third_party/re2/src
!/third_party/test262-harness
!/third_party/siphash
!/third_party/utf8-decoder
!/third_party/v8
!/third_party/valgrind
!/third_party/vtune
!/third_party/wasm-api
/tools/builtins-pgo/profiles/*
!/tools/builtins-pgo/profiles/.*
Expand Down Expand Up @@ -145,3 +152,4 @@ bazel-v8
!/third_party/fp16
/third_party/fp16/src/*
!/third_party/fp16/src/include
!/third_party/simdutf
10 changes: 9 additions & 1 deletion deps/v8/AUTHORS
Original file line number Diff line number Diff line change
Expand Up @@ -57,7 +57,7 @@ Alexander Botero-Lowry <[email protected]>
Alexander Karpinsky <[email protected]>
Alexander Neville <[email protected]>
Alexandre Vassalotti <[email protected]>
Alexey Pavlyutkin <alexey.pavlyutkin.community@gmail.com>
Alexey Pavlyutkin <alexey.pavlyutkin@syntacore.com>
Alexis Campailla <[email protected]>
Allan Sandfeld Jensen <[email protected]>
Amos Lim <[email protected]>
Expand Down Expand Up @@ -120,6 +120,7 @@ Dominic Farolini <[email protected]>
Douglas Crosher <[email protected]>
Dusan Milosavljevic <[email protected]>
Eden Wang <[email protected]>
Edoardo Marangoni <[email protected]>
Elisha Hollander <[email protected]>
Eric Rannaud <[email protected]>
Erich Ocean <[email protected]>
Expand Down Expand Up @@ -210,11 +211,13 @@ Maxim Mazurok <[email protected]>
Maxim Mossienko <[email protected]>
Md Hasibul Hasan <[email protected]>
Meir Shpilraien <[email protected]>
Meng Tan <[email protected]>
Michael Lutz <[email protected]>
Michael Mclaughlin <[email protected]>
Michael Smith <[email protected]>
Michaël Zasso <[email protected]>
Mihir Shah <[email protected]>
Mika Fischer <[email protected]>
Mike Gilbert <[email protected]>
Mike Pennisi <[email protected]>
Mikhail Gusarov <[email protected]>
Expand Down Expand Up @@ -268,6 +271,7 @@ Sébastien Doeraene <[email protected]>
Seo Sanghyeon <[email protected]>
Shawn Anastasio <[email protected]>
Shawn Presser <[email protected]>
Sho Miyamoto <[email protected]>
Stefan Penner <[email protected]>
Stefan Stojanovic <[email protected]>
Stephan Hartmann <[email protected]>
Expand All @@ -290,6 +294,7 @@ Varun Varada <[email protected]>
Victor Costan <[email protected]>
Victor Polevoy <[email protected]>
Vlad Burlik <[email protected]>
Vladimir Kempik <[email protected]>
Vladimir Krivosheev <[email protected]>
Vladimir Shutoff <[email protected]>
Wael Almattar <[email protected]>
Expand All @@ -303,6 +308,7 @@ Wiktor Garbacz <[email protected]>
Wouter Vermeiren <[email protected]>
Xiaofang Zou <[email protected]>
Xiaoyin Liu <[email protected]>
Yagiz Nizipli <[email protected]>
Yanbo Li <[email protected]>
Yannic Bonenberger <[email protected]>
Yi Wang <[email protected]>
Expand All @@ -317,6 +323,7 @@ Yuxiang Cao <[email protected]>
Zac Hansen <[email protected]>
Zeynep Cankara <[email protected]>
Zhao Jiazhong <[email protected]>
Zhao Qin <[email protected]>
Zhaojun Meng <[email protected]>
Zheng Liu <[email protected]>
Zhongping Wang <[email protected]>
Expand All @@ -325,3 +332,4 @@ Yang Xiang <[email protected]>
Kotaro Ohsugi <[email protected]>
Jing Peiyang <[email protected]>
magic-akari <[email protected]>
Ryuhei Shima <[email protected]>
Loading
Loading