Skip to content

Commit 7039a7a

Browse files
mpwarresleonm1
authored andcommitted
update V8 to 13.8.258.26
Signed-off-by: Michael Warres <[email protected]>
1 parent 6d91cbc commit 7039a7a

File tree

3 files changed

+11
-100
lines changed

3 files changed

+11
-100
lines changed

bazel/external/v8.patch

Lines changed: 8 additions & 41 deletions
Original file line numberDiff line numberDiff line change
@@ -1,13 +1,14 @@
11
# 1. Disable pointer compression (limits the maximum number of WasmVMs).
22
# 2. Don't expose Wasm C API (only Wasm C++ API).
3-
# 3. Fix gcc build error by disabling nonnull warning.
4-
# 4. Allow compiling v8 on macOS 10.15 to 13.0. TODO(dio): Will remove this patch when https://bugs.chromium.org/p/v8/issues/detail?id=13428 is fixed.
3+
# 3. Revert v8 commit b26554ec368e9553782012c96aa5e99b163eaff2, which removed
4+
# use of _allowlist_function_transition from v8 bazel/defs.bzl, since it is
5+
# still required by the version of Bazel we currently use (6.5.0).
56

67
diff --git a/BUILD.bazel b/BUILD.bazel
7-
index 4e89f90e7e..3fcb38b3f3 100644
8+
index 3f5a87d054e..0a693b7ee10 100644
89
--- a/BUILD.bazel
910
+++ b/BUILD.bazel
10-
@@ -157,7 +157,7 @@ v8_int(
11+
@@ -292,7 +292,7 @@ v8_int(
1112
# If no explicit value for v8_enable_pointer_compression, we set it to 'none'.
1213
v8_string(
1314
name = "v8_enable_pointer_compression",
@@ -16,45 +17,11 @@ index 4e89f90e7e..3fcb38b3f3 100644
1617
)
1718

1819
# Default setting for v8_enable_pointer_compression.
19-
diff --git a/bazel/defs.bzl b/bazel/defs.bzl
20-
index e957c0fad3..063627b72b 100644
21-
--- a/bazel/defs.bzl
22-
+++ b/bazel/defs.bzl
23-
@@ -131,6 +131,10 @@ def _default_args():
24-
"-Wno-redundant-move",
25-
"-Wno-return-type",
26-
"-Wno-stringop-overflow",
27-
+ "-Wno-nonnull",
28-
+ "-Wno-error=pessimizing-move",
29-
+ "-Wno-error=dangling-reference",
30-
+ "-Wno-error=dangling-pointer=",
31-
# Use GNU dialect, because GCC doesn't allow using
32-
# ##__VA_ARGS__ when in standards-conforming mode.
33-
"-std=gnu++17",
34-
@@ -151,6 +152,18 @@ def _default_args():
35-
"-fno-integrated-as",
36-
],
37-
"//conditions:default": [],
38-
+ }) + select({
39-
+ "@v8//bazel/config:is_macos": [
40-
+ # The clang available on macOS catalina has a warning that isn't clean on v8 code.
41-
+ "-Wno-range-loop-analysis",
42-
+
43-
+ # To supress warning on deprecated declaration on v8 code. For example:
44-
+ # external/v8/src/base/platform/platform-darwin.cc:56:22: 'getsectdatafromheader_64'
45-
+ # is deprecated: first deprecated in macOS 13.0.
46-
+ # https://bugs.chromium.org/p/v8/issues/detail?id=13428.
47-
+ "-Wno-deprecated-declarations",
48-
+ ],
49-
+ "//conditions:default": [],
50-
}),
51-
includes = ["include"],
52-
linkopts = select({
5320
diff --git a/src/wasm/c-api.cc b/src/wasm/c-api.cc
54-
index 4473e205c0..65a6ec7e1d 100644
21+
index 05e4029f183..d705be96a16 100644
5522
--- a/src/wasm/c-api.cc
5623
+++ b/src/wasm/c-api.cc
57-
@@ -2247,6 +2247,8 @@ auto Instance::exports() const -> ownvec<Extern> {
24+
@@ -2472,6 +2472,8 @@ WASM_EXPORT auto Instance::exports() const -> ownvec<Extern> {
5825

5926
} // namespace wasm
6027

@@ -63,7 +30,7 @@ index 4473e205c0..65a6ec7e1d 100644
6330
// BEGIN FILE wasm-c.cc
6431

6532
extern "C" {
66-
@@ -3274,3 +3276,5 @@ wasm_instance_t* wasm_frame_instance(const wasm_frame_t* frame) {
33+
@@ -3518,3 +3520,5 @@ wasm_instance_t* wasm_frame_instance(const wasm_frame_t* frame) {
6734
#undef WASM_DEFINE_SHARABLE_REF
6835

6936
} // extern "C"

bazel/external/v8_include.patch

Lines changed: 0 additions & 41 deletions
This file was deleted.

bazel/repositories.bzl

Lines changed: 3 additions & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -169,13 +169,12 @@ def proxy_wasm_cpp_host_repositories():
169169
maybe(
170170
git_repository,
171171
name = "v8",
172-
# 10.7.193.13
173-
commit = "6c8b357a84847a479cd329478522feefc1c3195a",
172+
# 13.8.258.26
173+
commit = "de9d0f8b56ae61896e4d2ac577fc589efb14f87d",
174174
remote = "https://chromium.googlesource.com/v8/v8",
175-
shallow_since = "1664374400 +0000",
175+
shallow_since = "1752074621 -0400",
176176
patches = [
177177
"@proxy_wasm_cpp_host//bazel/external:v8.patch",
178-
"@proxy_wasm_cpp_host//bazel/external:v8_include.patch",
179178
],
180179
patch_args = ["-p1"],
181180
)
@@ -185,20 +184,6 @@ def proxy_wasm_cpp_host_repositories():
185184
actual = "@v8//:wee8",
186185
)
187186

188-
maybe(
189-
new_git_repository,
190-
name = "com_googlesource_chromium_base_trace_event_common",
191-
build_file = "@v8//:bazel/BUILD.trace_event_common",
192-
commit = "521ac34ebd795939c7e16b37d9d3ddb40e8ed556",
193-
remote = "https://chromium.googlesource.com/chromium/src/base/trace_event/common.git",
194-
shallow_since = "1662508800 +0000",
195-
)
196-
197-
native.bind(
198-
name = "base_trace_event_common",
199-
actual = "@com_googlesource_chromium_base_trace_event_common//:trace_event_common",
200-
)
201-
202187
# WAMR with dependencies.
203188

204189
maybe(

0 commit comments

Comments
 (0)