[PATCH] D71695: [cmake] Include the llvm-locstats target when utils and tools are not being built.

David Spickett via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Dec 19 02:59:11 PST 2019


DavidSpickett added a comment.

In D71695#1790971 <https://reviews.llvm.org/D71695#1790971>, @djtodoro wrote:

> Sorry for the confusion. May be the `llvm/CMakeLists.txt` is better place to address this?
>
> llvm/CMakeLists.txt:
>
>   if (LLVM_INCLUDE_UTILS AND LLVM_INCLUDE_TOOLS)
>     add_subdirectory(utils/llvm-locstats)
>   endif()


Changing it to:

  if (LLVM_INCLUDE_UTILS AND LLVM_INCLUDE_TOOLS AND LLVM_BUILD_TOOLS)
    add_subdirectory(utils/llvm-locstats)

Doesn't help the issue, but maybe I'm missing your point here?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D71695





More information about the llvm-commits mailing list