Skip to content

Commit

Permalink
build: build v8 with -fvisibility=hidden
Browse files Browse the repository at this point in the history
This enables -fvisibility=hidden on platforms other than macOS,
wherever it's supported.
  • Loading branch information
joyeecheung committed Dec 17, 2024
1 parent 3b1bdd4 commit 3752a7e
Showing 1 changed file with 1 addition and 2 deletions.
3 changes: 1 addition & 2 deletions tools/v8_gypfiles/v8.gyp
Original file line number Diff line number Diff line change
Expand Up @@ -42,8 +42,7 @@
}
},
# Hide symbols that are not explicitly exported with V8_EXPORT.
# TODO(joyeecheung): enable it on other platforms. Currently gcc times out
# or run out of memory with -fvisibility=hidden on some machines in the CI.
'cflags': ['-fvisibility=hidden'],
'xcode_settings': {
'GCC_SYMBOLS_PRIVATE_EXTERN': 'YES', # -fvisibility=hidden
},
Expand Down

0 comments on commit 3752a7e

Please sign in to comment.