cmake: Rename installhdrs to install-llvm-headers and fix the dependencies
authorJustin Bogner <mail@justinbogner.com>
Mon, 24 Oct 2016 21:58:58 +0000 (21:58 +0000)
committerJustin Bogner <mail@justinbogner.com>
Mon, 24 Oct 2016 21:58:58 +0000 (21:58 +0000)
The installhdrs target was inconsistently named and would behave
differently depending on whether or not you ran a build first. This
renames it to install-llvm-headers to match other target names and
adds a dependency on intrinsics_gen so that it will always install the
same set of things.

llvm-svn: 285035

llvm/CMakeLists.txt

index cc6e583..2b51c97 100644 (file)
@@ -890,9 +890,13 @@ if (NOT LLVM_INSTALL_TOOLCHAIN_ONLY)
     PATTERN ".svn" EXCLUDE
     )
 
+  # Installing the headers needs to depend on generating any public
+  # tablegen'd headers.
+  add_custom_target(llvm-headers DEPENDS intrinsics_gen)
+
   if (NOT CMAKE_CONFIGURATION_TYPES)
-    add_custom_target(installhdrs
-                      DEPENDS ${name}
+    add_custom_target(install-llvm-headers
+                      DEPENDS llvm-headers
                       COMMAND "${CMAKE_COMMAND}"
                               -DCMAKE_INSTALL_COMPONENT=llvm-headers
                               -P "${CMAKE_BINARY_DIR}/cmake_install.cmake")