[PATCH] CMake: enable installing utils

Chris Bieneman beanz at apple.com
Wed Mar 25 14:14:32 PDT 2015


================
Comment at: cmake/modules/AddLLVM.cmake:554
@@ +553,3 @@
+      COMPONENT ${name})
+    add_custom_target(install-${name}
+                      DEPENDS ${name}
----------------
Sorry to nitpick, but you want to wrap this in:

if (NOT CMAKE_CONFIGURATION_TYPES)

That way you won't be creating an unwieldy number of extra targets for IDE users.

http://reviews.llvm.org/D8587

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/






More information about the llvm-commits mailing list