[PATCH] D54153: Fix compilation issue in VS2017 with Clang-tablegen and LLVM-tablegen

Tue Dec 18 17:31:35 PST 2018

aganea added a comment.

In D54153#1335403 <https://reviews.llvm.org/D54153#1335403>, @beanz wrote:

> It is expected that CMake will process all projects put under llvm/tools unless explicitly told not to. So, it is expected that it would process clang there. If you don't want it to process clang you need to set `LLVM_TOOL_CLANG_BUILD=Off` when configuring.

Ah, thank you for the explanation Chris.

> In terms of the error you're seeing. I'm not sure what is going on there, but the logged error doesn't make sense with the state of the code in tree today (line 165 of TableGen.cmake doesn't have a call to add_dependencies). Are you maybe building out of date sources?

No I reverted the change so I can investigate.

Repository:
  rL LLVM

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

https://reviews.llvm.org/D54153