[PATCH] D41402: [cmake] Fix DESTDIR support in compiler-rt build

Shoaib Meenai via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Dec 19 10:17:22 PST 2017


smeenai requested changes to this revision.
smeenai added a comment.
This revision now requires changes to proceed.

cmake will expand out relative paths for `PATH` cache variables, so the `.` will actually be expanded out to the build directory, which isn't desirable. You need to use the `STRING` type to avoid this normalization (though that's hacky).

Also, the canonical path is actually `$ENV{DESTDIR}${CMAKE_INSTALL_PREFIX}`; i.e., `DESTDIR` is supposed to contain the trailing slash itself.


Repository:
  rCRT Compiler Runtime

https://reviews.llvm.org/D41402





More information about the llvm-commits mailing list