commit 22fc4b612fa8cd0b6e62b693a804fb12651852cd
author David Spickett <david.spickett@arm.com> Thu Dec 19 15:00:56 2019 +0000
committer Copybara-Service <copybara-worker@google.com> Wed Oct 21 00:06:40 2020 -0700
tree 95f8d6677b5308198435699e2a7565d8d0dee2e6
parent 837698fbdd3b24009f72dd93aded87efebaa7dba

[cmake] Add dependency on llvm-dwarfdump to llvm-locstats

Though llvm-locstat is a "util" it requires llvm-dwarfdump
to be useful. So what happens now is that unless LLVM_BUILD_TOOLS
is ON llvm-locstats won't be part of the "all" target.

However given that it requires llvm-dwarfdump, this is less
confusing/build time consuming than suddenly having llvm-dwarfdump
build when all other tools are disabled.

Differential Revision: https://reviews.llvm.org/D71705

GitOrigin-RevId: ba430f503244d1498529d47f31090cdf79b5c231

utils/llvm-locstats/CMakeLists.txt

diff --git a/utils/llvm-locstats/CMakeLists.txt b/utils/llvm-locstats/CMakeLists.txt
index d5366f9..1dbb9da 100644
--- a/utils/llvm-locstats/CMakeLists.txt
+++ b/utils/llvm-locstats/CMakeLists.txt
@@ -2,11 +2,15 @@
   add_custom_command(
     OUTPUT ${LLVM_TOOLS_BINARY_DIR}/llvm-locstats
     DEPENDS ${LLVM_MAIN_SRC_DIR}/utils/llvm-locstats/llvm-locstats.py
+    DEPENDS llvm-dwarfdump
     COMMAND ${CMAKE_COMMAND} -E copy ${LLVM_MAIN_SRC_DIR}/utils/llvm-locstats/llvm-locstats.py ${LLVM_TOOLS_BINARY_DIR}/llvm-locstats
     COMMENT "Copying llvm-locstats into ${LLVM_TOOLS_BINARY_DIR}"
     )
   add_custom_target(llvm-locstats ALL
     DEPENDS ${LLVM_TOOLS_BINARY_DIR}/llvm-locstats
     )
+  if (NOT LLVM_BUILD_TOOLS)
+    set_target_properties(llvm-locstats PROPERTIES EXCLUDE_FROM_ALL ON)
+  endif()
   set_target_properties(llvm-locstats PROPERTIES FOLDER "Tools")
 endif()