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

Saleem Abdulrasool via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jan 31 10:07:08 PST 2022


compnerd 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()
----------------
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.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D118186



More information about the llvm-commits mailing list