[PATCH] D129797: [clang-tidy] Reduce the dependencies for the "make-confusable-table" tool

Martin Storsjö via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Jul 14 13:25:18 PDT 2022


mstorsjo created this revision.
mstorsjo added reviewers: serge-sans-paille, sammccall, whisperity, aaron.ballman.
Herald added subscribers: carlosgalvezp, rnkovacs, xazax.hun, mgorny.
Herald added a project: All.
mstorsjo requested review of this revision.
Herald added a project: clang-tools-extra.

When cross compiling llvm, a separate recursive native cmake build
is generated, for building the tools that generate code (unless they're
provided externally by the caller).

This reduces the number of build steps for that native build from
1000+ steps to 162.

This matches how the clang-pseudo-gen tool is set up in
clang-tools-extra/pseudo/gen/CMakeLists.txt.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D129797

Files:
  clang-tools-extra/clang-tidy/misc/ConfusableTable/CMakeLists.txt


Index: clang-tools-extra/clang-tidy/misc/ConfusableTable/CMakeLists.txt
===================================================================
--- clang-tools-extra/clang-tidy/misc/ConfusableTable/CMakeLists.txt
+++ clang-tools-extra/clang-tidy/misc/ConfusableTable/CMakeLists.txt
@@ -1,3 +1,6 @@
+set(LLVM_LINK_COMPONENTS Support)
+list(REMOVE_ITEM LLVM_COMMON_DEPENDS clang-tablegen-targets)
+
 add_llvm_executable(make-confusable-table
   BuildConfusableTable.cpp
   )


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D129797.444778.patch
Type: text/x-patch
Size: 469 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20220714/252fc40a/attachment.bin>


More information about the cfe-commits mailing list