[PATCH] D71705: [cmake] Add dependency on llvm-dwarfdump to llvm-locstats

[Djordje Todorovic via Phabricator via llvm-commits]

djtodoro accepted this revision.
djtodoro added a comment.
This revision is now accepted and ready to land.

Thanks for doing this! Looks good to me now!



================
Comment at: llvm/utils/llvm-locstats/CMakeLists.txt:12
     )
+  if (NOT LLVM_BUILD_UTILS)
+    set_target_properties(llvm-locstats PROPERTIES EXCLUDE_FROM_ALL ON)
----------------
DavidSpickett wrote:
> djtodoro wrote:
> > Please describe, once again, why you decided to use the `LLVM_BUILD_UTILS` instead of the `LLVM_BUILD_TOOLS` here.
> I was testing both and got confused along the way. LLVM_BUILD_TOOLS is what I intended.
> 
> Reasoning being that though llvm-locstats requires llvm-dwarfdump, it would be a bigger problem to start building llvm-dwarfdump when LLVM_BUILD_TOOLS is OFF, than it is to not build llvm-locstats when LLVM_BUILD_UTILS is ON.
> 
> (from the point of view of someone trying to trim their build time)
That is why I asked this. :)


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D71705/new/

https://reviews.llvm.org/D71705