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

[David Tenty via Phabricator via llvm-commits]

daltenty added a comment.

LGTM, thanks

Context: the  tool chain doesn't support modifying rpaths/libpaths for XCOFF on install at the moment, so this is required.



================
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 "")
----------------
nit: let's update the comment



Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D148866