Skip to content

Hide non-public symbols by default#3381

Draft
ttnghia wants to merge 1 commit intoNVIDIA:mainfrom
ttnghia:hide_symbols
Draft

Hide non-public symbols by default#3381
ttnghia wants to merge 1 commit intoNVIDIA:mainfrom
ttnghia:hide_symbols

Conversation

@ttnghia
Copy link
Collaborator

@ttnghia ttnghia commented May 28, 2025

This set CXX_VISIBILITY_PRESET and CUDA_VISIBILITY_PRESET to hidden, hiding non-public symbols by default to avoid non-API usage and potential conflicts with symbols from dependencies.

Closes #2151.

Signed-off-by: Nghia Truong <nghiat@nvidia.com>
@ttnghia ttnghia self-assigned this May 28, 2025
@ttnghia ttnghia added the build label May 28, 2025
@ttnghia
Copy link
Collaborator Author

ttnghia commented May 28, 2025

build

@ttnghia ttnghia linked an issue May 28, 2025 that may be closed by this pull request
@ttnghia ttnghia marked this pull request as draft May 28, 2025 22:48
@ttnghia ttnghia changed the base branch from branch-25.08 to branch-25.10 August 4, 2025 14:29
@ttnghia ttnghia changed the base branch from branch-25.10 to branch-25.12 September 29, 2025 17:41
@nvauto
Copy link
Collaborator

nvauto commented Nov 17, 2025

NOTE: release/25.12 has been created from main. Please retarget your PR to release/25.12 if it should be included in the release.

@nvauto
Copy link
Collaborator

nvauto commented Jan 19, 2026

NOTE: release/26.02 has been created from main. Please retarget your PR to release/26.02 if it should be included in the release.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[FEA] Limit symbol visibility in shared library

2 participants