Skip to content

Commit 8bb84d6

Browse files
committed
build: disable libstdc++ debug containers globally
Different parts of the debug build were using differently sized std::vectors due to `_GLIBCXX_DEBUG` sometimes being defined and sometimes not. That ended about as well as you would expect. Remove the flag. Fixes: nodejs#30056
1 parent 4bec6d1 commit 8bb84d6

File tree

2 files changed

+0
-12
lines changed

2 files changed

+0
-12
lines changed

common.gypi

-4
Original file line numberDiff line numberDiff line change
@@ -30,10 +30,6 @@
3030
'openssl_fips%': '',
3131
'openssl_no_asm%': 0,
3232

33-
# Some STL containers (e.g. std::vector) do not preserve ABI compatibility
34-
# between debug and non-debug mode.
35-
'disable_glibcxx_debug': 1,
36-
3733
# Don't use ICU data file (icudtl.dat) from V8, we use our own.
3834
'icu_use_data_file_flag%': 0,
3935

tools/v8_gypfiles/toolchain.gypi

-8
Original file line numberDiff line numberDiff line change
@@ -64,9 +64,6 @@
6464
# Print to stdout on Android.
6565
'v8_android_log_stdout%': 0,
6666

67-
# Force disable libstdc++ debug mode.
68-
'disable_glibcxx_debug%': 0,
69-
7067
'v8_enable_backtrace%': 0,
7168

7269
# Enable profiling support. Only required on Windows.
@@ -1167,11 +1164,6 @@
11671164
# Support for backtrace_symbols.
11681165
'ldflags': [ '-rdynamic' ],
11691166
}],
1170-
['OS=="linux" and disable_glibcxx_debug==0', {
1171-
# Enable libstdc++ debugging facilities to help catch problems
1172-
# early, see http://crbug.com/65151 .
1173-
'defines': ['_GLIBCXX_DEBUG=1',],
1174-
}],
11751167
['OS=="aix"', {
11761168
'ldflags': [ '-Wl,-bbigtoc' ],
11771169
'conditions': [

0 commit comments

Comments
 (0)