[PATCH] [CMake] Fix installation without CLANG_BUILD_EXAMPLES

Jordan Rose jordan_rose at apple.com
Thu Feb 20 09:49:25 PST 2014


On Feb 20, 2014, at 9:35 , Brad King <brad.king at kitware.com> wrote:

> On 02/20/2014 12:06 PM, Jordan Rose wrote:
>> Nice catch. What made you decide to do this rather than change AddLLVM.cmake?
> 
> AFAICT this approach uses the AddLLVM APIs as designed.  They already
> know not to install targets when EXCLUDE_FROM_ALL is enabled but were
> not told about the setting in this case within Clang.

Seems fair—there's not a great way to pass that down to add_target otherwise. Committed as r201799.

Thanks!
Jordan





More information about the cfe-commits mailing list