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

Mehdi AMINI via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Mar 3 14:52:35 PST 2023


mehdi_amini added a comment.

In D145258#4168143 <https://reviews.llvm.org/D145258#4168143>, @dblaikie wrote:

> I would've thought it was necessary to alwayslink them because maybe they use some kind of auto-registration system into clang-tidy, so without alwayslink your clang-tidy binary wouldn't link any of them in? But I could be wrong, maybe they're explicitly referenced in the clang-tidy binary?

There is no always-link in cmake as far as I know, so we should not need any in Bazel either in the LLVM project hopefully.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D145258



More information about the llvm-commits mailing list