[PATCH] D55001: [CMake] Fix build with -DLLVM_TOOL_LLVM_{MCA/EXEGESIS}_BUILD=OFF

Michael Platings via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Nov 29 02:32:01 PST 2018


michaelplatings added a comment.

Out of interest, how are you disabling tools?

Just an idea, but how about making all the targets optional? It's less code and gives you more flexibility should you wish to disable more of them in future.

  set(LLVM_TEST_DEPENDS_CANDIDATES
            BugpointPasses
  [...]
            yaml2obj
          )
  
  foreach(candidate ${LLVM_TEST_DEPENDS_CANDIDATES})
    if(TARGET ${candidate})
      list(APPEND LLVM_TEST_DEPENDS ${candidate})
    endif()
  endforeach()


Repository:
  rL LLVM

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

https://reviews.llvm.org/D55001





More information about the llvm-commits mailing list