tools: update V8 gypfiles for 8.7 · nodejs/node@770d9e2 · GitHub
Skip to content

Commit 770d9e2

Browse files
targostargos
committed
tools: update V8 gypfiles for 8.7
PR-URL: #35700 Reviewed-By: Rich Trott <rtrott@gmail.com> Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Joyee Cheung <joyeec9h3@gmail.com> Reviewed-By: Matteo Collina <matteo.collina@gmail.com> Reviewed-By: Jiawen Geng <technicalcute@gmail.com> Reviewed-By: Shelley Vohr <codebytere@gmail.com>
1 parent 6f9cbcf commit 770d9e2

3 files changed

Lines changed: 39 additions & 45 deletions

File tree

‎tools/v8_gypfiles/features.gypi‎

Lines changed: 0 additions & 6 deletions

‎tools/v8_gypfiles/toolchain.gypi‎

Lines changed: 1 addition & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -86,9 +86,6 @@
8686
# For a shared library build, results in "libv8-<(soname_version).so".
8787
'soname_version%': '',
8888

89-
# Allow to suppress the array bounds warning (default is no suppression).
90-
'wno_array_bounds%': '',
91-
9289
# Override where to find binutils
9390
'binutils_dir%': '',
9491

@@ -141,7 +138,7 @@
141138
'cflags': [ '-Werror', '-Wno-unknown-pragmas' ],
142139
},{
143140
'cflags!': [ '-Wall', '-Wextra' ],
144-
'cflags': [ '-Wno-return-type' ],
141+
'cflags': [ '-Wno-return-type', '-Wno-int-in-bool-context' ],
145142
}],
146143
['v8_target_arch=="arm"', {
147144
'defines': [
@@ -1116,10 +1113,6 @@
11161113
'V8_ENABLE_FORCE_SLOW_PATH',
11171114
],
11181115
'conditions': [
1119-
['OS=="linux" or OS=="freebsd" or OS=="openbsd" or OS=="netbsd" or \
1120-
OS=="qnx" or OS=="aix"', {
1121-
'cflags': [ '-Woverloaded-virtual', '<(wno_array_bounds)', ],
1122-
}],
11231116
['OS=="linux" and v8_enable_backtrace==1', {
11241117
# Support for backtrace_symbols.
11251118
'ldflags': [ '-rdynamic' ],
@@ -1276,7 +1269,6 @@
12761269
'cflags': [
12771270
'-fdata-sections',
12781271
'-ffunction-sections',
1279-
'<(wno_array_bounds)',
12801272
],
12811273
'conditions': [
12821274
# Don't use -O3 with sanitizers.

‎tools/v8_gypfiles/v8.gyp‎

Lines changed: 38 additions & 30 deletions

0 commit comments

Comments
 (0)