Skip to content

SWDEV-428379 - Set LD_LIBRARY_PATH for executing the binaries from build directory #449

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 3 commits into from
Jan 8, 2024

Conversation

raramakr
Copy link
Contributor

The binaries rocfft_aot_helper and stockham_aot are getting executed from build directory during build time. The hardcoded RUNPATH in this binaries are helping to find the required libraries. Hardcoded RUNPATH will be removed and will be having relative RUNPATH with respect to install directory.

LD_LIBRARY_PATH need to set during build time for a successful execution of the binaries

resolves #___

Summary of proposed changes:

…ild directory

The binaries rocfft_aot_helper and stockham_aot are getting executed from build directory during build time. The hardcoded RUNPATH in this binaries are helping to find the required libraries. Hardcoded RUNPATH will be removed and will be having relative RUNPATH with respect to install directory.

LD_LIBRARY_PATH need to set during build time for a successfull exectuion of the binaries
In ASAN build, the environment variable LD_LIBRARY_PATH is used to set libclang_rt.asan.so library path.
Use the LD_LIBRARY_PATH provided by build script
@raramakr raramakr requested a review from nunnikri December 13, 2023 23:07
Since the binary is installed in /opt/rocm-ver/lib/rocfft/version_string folder, it need relative RPATH $ORIGIN/../../../lib to find rocm libraries
@ROCmMathLibrariesBot
Copy link

Performance reports:
Commit hashes:
e84424d
gfx90a single report
gfx90a double report

@raramakr raramakr merged commit 0ec78f1 into ROCm:release/rocm-rel-6.0 Jan 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants