[PATCH] D146352: Introduce `llvm-min-tblgen` to build public header files

[Martin Storsjö via Phabricator via llvm-commits]

mstorsjo added a comment.

I think this looks quite neat overall, good stuff!

I measured the benefit from this patch in another build case; where I just want to build a small set of executables (for testing compiller-rt) in a new build tree; building tools like `FileCheck` is quite quick in general, but `llvm-config` has got surprisingly large numbers of dependencies. With this patch, the number of build steps to build `llvm-config` drops from 248 to 188, and the total cpu time for compiling it drops from ~11 minutes to ~5 minutes.



================
Comment at: llvm/cmake/modules/TableGen.cmake:185
+      add_custom_target(${target}-host DEPENDS ${${project}_TABLEGEN_EXE})
+      set(${project}_TABLEGEN_TARGET ${target}-host PARENT_SCOPE)
 
----------------
I guess these two lines could be consider a standalone NFC refactoring/cleanup on top of the current code as well?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D146352