Add @loader_path to rpath on mac

[dschuff]

Summary

Currently @executable_path/../lib is used as the rpath for mac builds.
When using LLVM_LINK_LLVM_DYLIB, libLTO.dylib is linked against libLLVM.dylib,
just like all the executables. However libLTO fails to load because in this
case the executable is ld (instead of one of the LLVM tools) which is in a
different location. Adding @loader_path to the rpath list fixes this
problem.

[dschuff]

So this seems to work but I'm not sure if it's the best solution; I'm hoping an OSX and/or CMake expert can help. Should I make it conditional on -DLLVM_LINK_LLVM_DYLIB? Can/should it be done just for libLTO and not for the executable tools? Something else?

[dschuff]

• ../lib is redundant

[beanz]

There is actually a refactoring of this code being reviewed at the moment, which should address this issue. See D25304.

[dschuff]

Great, thanks! I'll try that one then.

[dschuff]

Fixed in rL285714, thanks!

CMakeLists.txt

 # They are not referenced. See set_output_directory().
 set( CMAKE_RUNTIME_OUTPUT_DIRECTORY ${LLVM_BINARY_DIR}/bin )
 set( CMAKE_LIBRARY_OUTPUT_DIRECTORY ${LLVM_BINARY_DIR}/lib${LLVM_LIBDIR_SUFFIX} )
 set( CMAKE_ARCHIVE_OUTPUT_DIRECTORY ${LLVM_BINARY_DIR}/lib${LLVM_LIBDIR_SUFFIX} )
 
 set(CMAKE_BUILD_WITH_INSTALL_RPATH ON)
 if (APPLE)
   set(CMAKE_INSTALL_NAME_DIR "@rpath")
-  set(CMAKE_INSTALL_RPATH "@executable_path/../lib")
+  set(CMAKE_INSTALL_RPATH "@executable_path/../lib;@loader_path/../lib")
 else(UNIX)
   if(NOT DEFINED CMAKE_INSTALL_RPATH)
     set(CMAKE_INSTALL_RPATH "\$ORIGIN/../lib${LLVM_LIBDIR_SUFFIX}")
     if(${CMAKE_SYSTEM_NAME} MATCHES "(FreeBSD|DragonFly)")
       set(CMAKE_EXE_LINKER_FLAGS "${CMAKE_EXE_LINKER_FLAGS} -Wl,-z,origin")
       set(CMAKE_SHARED_LINKER_FLAGS "${CMAKE_SHARED_LINKER_FLAGS} -Wl,-z,origin")
     endif()
   endif(NOT DEFINED CMAKE_INSTALL_RPATH)