From b63109944d3ef77b3b4a39eaf1908edfe2fbb8cb Mon Sep 17 00:00:00 2001 From: Justin Bogner Date: Mon, 24 Oct 2016 21:58:58 +0000 Subject: [PATCH] cmake: Rename installhdrs to install-llvm-headers and fix the dependencies 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 | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/llvm/CMakeLists.txt b/llvm/CMakeLists.txt index cc6e583..2b51c97 100644 --- a/llvm/CMakeLists.txt +++ b/llvm/CMakeLists.txt @@ -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") -- 2.7.4