[PATCH] D145258: [bazel] Don't alwayslink clang-tidy libraries

Aaron Siddhartha Mondal via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Mar 3 10:57:29 PST 2023


aaronmondal created this revision.
Herald added a subscriber: carlosgalvezp.
Herald added a reviewer: njames93.
Herald added a project: All.
aaronmondal requested review of this revision.
Herald added a project: LLVM.
Herald added a subscriber: llvm-commits.

These libraries are only ever used in clang-tidy itself, so there is no
need to unconditionally keep all symbols.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D145258

Files:
  utils/bazel/llvm-project-overlay/clang-tools-extra/clang-tidy/defs.bzl


Index: utils/bazel/llvm-project-overlay/clang-tools-extra/clang-tidy/defs.bzl
===================================================================
--- utils/bazel/llvm-project-overlay/clang-tools-extra/clang-tidy/defs.bzl
+++ utils/bazel/llvm-project-overlay/clang-tools-extra/clang-tidy/defs.bzl
@@ -21,6 +21,5 @@
     kwargs["deps"] = kwargs.get("deps", []) + _common_library_deps
     cc_library(
         name = name,
-        alwayslink = True,
         **kwargs
     )


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D145258.502188.patch
Type: text/x-patch
Size: 474 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20230303/14c8656c/attachment.bin>


More information about the llvm-commits mailing list