[PATCH] D118186: [cmake] Allow optionally to build `tablegen` in `Release` and use it in `Debug` for the `Ninja Multi-Config` generator

[Argyrios Kyrtzidis via Phabricator via llvm-commits]

akyrtzi added inline comments.


================
Comment at: llvm/cmake/modules/TableGen.cmake:103
+    # we'll build for both release and debug.
+    set(tablegen_depends ${tablegen_exe})
+  endif()
----------------
compnerd wrote:
> This can actually break the dependency tracking; I think that we should figure out how to get the target setup and use that.  At that point, rather than computing the path manually, we should be able to use the generator expression `$<TARGET_FILE:target>` to get the path.
Could you suggest a specific change? Unfortunately my CMake-foo is rather weak..😅


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D118186