This is an archive of the discontinued LLVM Phabricator instance.

[gn] Fix clang-tidy build
ClosedPublic

Authored by mbonadei on Feb 5 2019, 12:01 PM.

Details

Summary

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)

Diff Detail

Event Timeline

mbonadei created this revision.Feb 5 2019, 12:01 PM
Herald added a project: Restricted Project. · View Herald TranscriptFeb 5 2019, 12:01 PM
thakis accepted this revision.Feb 5 2019, 3:45 PM
This revision is now accepted and ready to land.Feb 5 2019, 3:45 PM
This revision was automatically updated to reflect the committed changes.