[clang-tools-extra] [clang-tidy] Remove redundant LINK_LIBS (PR #94588)

Nikita Popov via cfe-commits cfe-commits at lists.llvm.org
Thu Jun 6 01:32:42 PDT 2024


https://github.com/nikic created https://github.com/llvm/llvm-project/pull/94588

clangAnalysis is already being pulled in via clang_target_link_libraries(). Also listing it in LINK_LIBS means that we'll link both against the static libraries and the shared libclang-cpp.so library if CLANG_LINK_CLANG_DYLIB is enabled, and waste time on unnecessary LTO.

>From b4dda7214c4c6431c9f978296f68b494c4fc7b68 Mon Sep 17 00:00:00 2001
From: Nikita Popov <npopov at redhat.com>
Date: Thu, 6 Jun 2024 10:25:25 +0200
Subject: [PATCH] [clang-tidy] Remove redundant LINK_LIBS

clangAnalysis is already being pulled in via
clang_target_link_libraries(). Also listing it in LINK_LIBS means
that we'll link both against the static libraries and the shared
libclang-cpp.so library if CLANG_LINK_CLANG_DYLIB is enabled, and
waste time on unnecessary LTO.
---
 clang-tools-extra/clang-tidy/misc/CMakeLists.txt | 1 -
 1 file changed, 1 deletion(-)

diff --git a/clang-tools-extra/clang-tidy/misc/CMakeLists.txt b/clang-tools-extra/clang-tidy/misc/CMakeLists.txt
index 35e29b9a7d13..36fcd8fc1b27 100644
--- a/clang-tools-extra/clang-tidy/misc/CMakeLists.txt
+++ b/clang-tools-extra/clang-tidy/misc/CMakeLists.txt
@@ -43,7 +43,6 @@ add_clang_library(clangTidyMiscModule
   UseAnonymousNamespaceCheck.cpp
 
   LINK_LIBS
-  clangAnalysis
   clangTidy
   clangTidyUtils
 



More information about the cfe-commits mailing list