[PATCH] D41244: Added a separate install target for compilert-rt-headers

Ilya Biryukov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Dec 15 02:38:25 PST 2017


ilya-biryukov planned changes to this revision.
ilya-biryukov added a comment.

In https://reviews.llvm.org/D41244#955511, @smeenai wrote:

> Isn't this gonna break building compiler-rt standalone (i.e. without LLVM)? Or does the include/ directory already rely on LLVM in some other way? See the history of https://reviews.llvm.org/D40687.


I was not aware of this use-case, thanks for mentioning it. It would definitely break without LLVM.
I'll update the patch to not use `add_llvm_install_targets`.


Repository:
  rCRT Compiler Runtime

https://reviews.llvm.org/D41244





More information about the llvm-commits mailing list