[PATCH] D130586: [cmake] Use `CMAKE_INSTALL_LIBDIR` too
John Ericson via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Wed Jul 27 08:41:25 PDT 2022
Ericson2314 marked 6 inline comments as done.
Ericson2314 added inline comments.
================
Comment at: compiler-rt/cmake/Modules/CompilerRTAIXUtils.cmake:65
set(output_dir "${LLVM_LIBRARY_OUTPUT_INTDIR}")
- set(install_dir "${CMAKE_INSTALL_PREFIX}/lib${LLVM_LIBDIR_SUFFIX}")
+ set(install_dir "${CMAKE_INSTALL_PREFIX}/lib")
else()
----------------
Ericson2314 wrote:
> sebastian-ne wrote:
> > This is an install directory, so should this be something like
> > ```
> > extend_path(install_dir "${CMAKE_INSTALL_PREFIX}" "${CMAKE_INSTALL_LIBDIR}")
> > ```
> > instead?
> Indeed, thanks for catching!
>
> `${CMAKE_INSTALL_FULL_LIBDIR}` also does this.
Actually, it will do the `CMAKE_INSTALL_PREFIX` part by default (and the other branch is already a potentially relative path) so just `${CMAKE_INSTALL_LIBDIR}` will suffice.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D130586/new/
https://reviews.llvm.org/D130586
More information about the cfe-commits
mailing list