replica
/
hanchenye-llvm-project
2
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

gn build: Fix clang-tidy build 

Not depending on //clang/lib/StaticAnalyzer/Core and
//clang/lib/StaticAnalyzer/Frontend causes a linker error even if
ClangSACheckers are not supported.

Undefined symbols for architecture x86_64:
  "clang::ento::CreateAnalysisConsumer(clang::CompilerInstance&)", referenced from:
      clang::tidy::ClangTidyASTConsumerFactory::CreateASTConsumer(
              clang::CompilerInstance&, llvm::StringRef)
          in libclangTidy.a(libclangTidy.ClangTidy.o)

Patch from Mirko Bonadei <mbonadei@webrtc.org>!

Differential Revision: https://reviews.llvm.org/D57777

llvm-svn: 353244
This commit is contained in:
Nico Weber 2019-02-05 23:48:13 +00:00
parent 052f7ff96a
commit d3391bdd91
1 changed files with 6 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

14
llvm/utils/gn/secondary/clang-tools-extra/clang-tidy/BUILD.gn
View File

@ -22,14 +22,12 @@ static_library("clang-tidy") {
# ClangSACheckers # ClangSACheckers
] ]
# TODO(mbonadei): Add support for Clang Static Analyzer checkers. if (clang_enable_static_analyzer) {
# Without this, "clang-analyzer-" will not work. deps += [
# if (clang_enable_static_analyzer) { "//clang/lib/StaticAnalyzer/Core",
# deps += [ "//clang/lib/StaticAnalyzer/Frontend",
# "//clang/lib/StaticAnalyzer/Core", ]
# "//clang/lib/StaticAnalyzer/Frontend", }
# ]
# }
sources = [ sources = [
"ClangTidy.cpp", "ClangTidy.cpp",