[PATCH] D148866: [CMake][AIX] Fixing AIX rpath

Qiongsi Wu via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Apr 21 06:48:42 PDT 2023


qiongsiwu1 marked an inline comment as done.
qiongsiwu1 added inline comments.


================
Comment at: llvm/cmake/modules/AddLLVM.cmake:2358
   # Enable BUILD_WITH_INSTALL_RPATH unless CMAKE_BUILD_RPATH is set and not
   # building for macOS or Windows, as those two platforms seemingly require it.
   if("${CMAKE_BUILD_RPATH}" STREQUAL "")
----------------
daltenty wrote:
> nit: let's update the comment
> 
Ah thanks for catching this David! Fixed. 


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D148866/new/

https://reviews.llvm.org/D148866



More information about the llvm-commits mailing list