This is an archive of the discontinued LLVM Phabricator instance.

[clang-tidy] Fix missing dependency to static analyzer
ClosedPublic

Authored by etienneb on Aug 2 2016, 3:44 PM.

Reviewers

Commits

Summary

This patch is fixing the build bot broken for a missing dependency.

The missing dependency is breaking "shared" build.

./mpi/TypeMismatchCheck.cpp:
  #include "clang/StaticAnalyzer/Checkers/MPIFunctionClassifier.h"

I'm not a fan of that fix. It's brining a strange dependency.

etienneb retitled this revision from to [clang-tidy] Fix missing dependency to static analyzer.

etienneb updated this object.

etienneb added a reviewer: alexfh.

This revision was automatically updated to reflect the committed changes.

Awesome, thanks!

Path

Size

clang-tools-extra/

trunk/

clang-tidy/

mpi/

1 line

	set(LLVM_LINK_COMPONENTS support)			set(LLVM_LINK_COMPONENTS support)

	add_clang_library(clangTidyMPIModule			add_clang_library(clangTidyMPIModule
	MPITidyModule.cpp			MPITidyModule.cpp
	TypeMismatchCheck.cpp			TypeMismatchCheck.cpp

	LINK_LIBS			LINK_LIBS
	clangAST			clangAST
	clangASTMatchers			clangASTMatchers
	clangBasic			clangBasic
	clangLex			clangLex
	clangTidy			clangTidy
	clangTidyUtils			clangTidyUtils
	clangTooling			clangTooling
				clangStaticAnalyzerCheckers
	)			)