[PATCH] D14502: [PATCH] Do not create a clang-install target for MSVC solutions
Chris Bieneman via cfe-commits
cfe-commits at lists.llvm.org
Mon Nov 9 14:18:35 PST 2015
> On Nov 9, 2015, at 2:07 PM, Aaron Ballman <aaron.ballman at gmail.com> wrote:
>
> aaron.ballman added inline comments.
>
> ================
> Comment at: tools/driver/CMakeLists.txt:58
> @@ -62,1 +57,3 @@
>
> +if(NOT MSVC)
> + add_custom_target(install-clang
> ----------------
> beanz wrote:
>> Can you change this to `if(NOT CMAKE_CONFIGURATION_TYPES)`?
>>
>> With that it won't create the install-clang target for any IDE, which is probably the way I should have done this in the first place.
>>
>> Other than that LGTM.
>>
>> Thanks,
>> -Chris
> Thanks, I'll make that modification. Out of curiosity, do you think INSTALL and PACKAGE should be opt-in for MSVC? Not present at all? Always present (as they are currently)?
Not sure I’m really a good person to comment on MSVC as I don’t use it. In general I don’t see harm in it, but this case of having the install-clang target was really just a bug. The purpose of that target is really to provide a convenience mechanism for people using Ninja and Make to only install clang.
Thank you for fixing this,
-Chris
>
>
> http://reviews.llvm.org/D14502
>
>
>
More information about the cfe-commits
mailing list