OSDN Git Service

[CMake] Fix incorrect rpath for tests if LLVM_LOCAL_RPATH isn't set
authorRaul Tambre <raul@tambre.ee>
Fri, 8 Jan 2021 19:19:42 +0000 (21:19 +0200)
committerRaul Tambre <raul@tambre.ee>
Fri, 8 Jan 2021 19:24:18 +0000 (21:24 +0200)
commit840a2c978374ccc74a0e0b2576c1027fdb609a59
treeb7704acf5e82c897f50f43285e02c9c47e21f783
parentbf401256edd00e921a5d3a0bf4cf6ee66ae51cd6
[CMake] Fix incorrect rpath for tests if LLVM_LOCAL_RPATH isn't set

d9ce31ae7d (D94322) removed the check because I thought it was dead
due to checking the existance of a variable (which always existed).

This causes LLDB tests to fail as they set NO_INSTALL_RPATH because
they're never meant to be installed, but we still would end up using
the install rpath.

Add the check back and make it explicitly check for an empty value
to make the purpose clearer and avoid implicit test for a false/true
value.

Differential Revision: https://reviews.llvm.org/D94326
llvm/cmake/modules/AddLLVM.cmake