[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:04:56 PST 2015


beanz accepted this revision.
This revision is now accepted and ready to land.

================
Comment at: tools/driver/CMakeLists.txt:58
@@ -62,1 +57,3 @@
 
+if(NOT MSVC)
+  add_custom_target(install-clang
----------------
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


http://reviews.llvm.org/D14502





More information about the cfe-commits mailing list