KhronosGroup / KhronosGroup/SPIRV-LLVM-Translator

expose less symbols by building with -fvisibility=hidden

Open
#1,963 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
LLVM
Stars
625
Forks
279
Avg merge
3d 5h
Merged PRs (30d)
34

Description

Hi,

it would be nice if the `libLLVMSPIRVLib.so.X` library could be built with `-fvisibility=hidden` s.t. it doesn't expose tons of internal C++ symbols, see https://gcc.gnu.org/wiki/Visibility for more details.

A (preliminary) patch can be found in the Debian package: https://salsa.debian.org/opencl-team/spirv-llvm-translator/-/blob/llvm15/main/debian/patches/visibility-hidden.patch

Contributor guide

Open the contributing guide

Research direction

Review the Debian package's visibility-hidden.patch linked in the issue and identify the corresponding library build configuration for libLLVMSPIRVLib.so.X. Verify that the completed build uses -fvisibility=hidden and that internal C++ symbols are no longer broadly exported.

Written by the indexing model from the issue text.

Assessment

Tech stack
cpp
Domain
build-system
Issue type
Feature
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.