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

Geoffrey Martin-Noble via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Mar 6 12:17:50 PST 2023


GMNGeoffrey added a comment.

In D145258#4172488 <https://reviews.llvm.org/D145258#4172488>, @GMNGeoffrey wrote:

> Hmmm actually our internal version of these build files also uses `alwayslink` with the comment "Registration of the clang-tidy checker modules relies on the linker, so we need to enforce linking". That comment was written in 2014 though, so things may have changed. Let me try seeing what tests fail if I remove that internally.

I tried removing it internally and nothing fails, so I think this is vestigial. Let's kill it!


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