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

Mon Mar 6 07:35:24 PST 2023

dblaikie added a comment.

In D145258#4168410 <https://reviews.llvm.org/D145258#4168410>, @mehdi_amini wrote:

> 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.

Ah, fair enough. It was just a vague guess at why this might've been included in the first place. If it doesn't apply - no worries.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D145258