This is an archive of the discontinued LLVM Phabricator instance.

Differential D65462

gn build: Fix check-clang-tools after r362702.
ClosedPublic

Authored by thakis on Jul 30 2019, 11:07 AM.

Details

Reviewers
hans
pcc
mbonadei
jdoerfert
Commits
rG372334b3d334: Merging r367340 and r367341:
rL367390: Merging r367340 and r367341:
rG2859bbb3e26e: gn build: Fix check-clang-tools after r362702.
rL367340: gn build: Fix check-clang-tools after r362702.
Summary

r362702 added a test that requires clang-tidy to be linked
into libclang, so add that to the gn build.

Diff Detail

Repository
rL LLVM

Revision Contents

PathSize
llvm/
trunk/
utils/
gn/
secondary/
clang-tools-extra/
clang-include-fixer/
plugin/
18 lines
clang-tidy/
plugin/
4 lines
clang/
tools/
libclang/
16 lines

Diff 212391

llvm/trunk/utils/gn/secondary/clang-tools-extra/clang-include-fixer/plugin/BUILD.gn

static_library("plugin") {
output_name = "clangIncludeFixerPlugin"
configs += [ "//llvm/utils/gn/build:clang_code" ]
deps = [
"//clang-tools-extra/clang-include-fixer",
"//clang/lib/AST",
"//clang/lib/Basic",
"//clang/lib/Frontend",
"//clang/lib/Parse",
"//clang/lib/Sema",
"//clang/lib/Tooling",
"//llvm/utils/gn/build/libs/pthread",
]
sources = [
"IncludeFixerPlugin.cpp",
]
}

llvm/trunk/utils/gn/secondary/clang-tools-extra/clang-tidy/plugin/BUILD.gn

import("//clang/lib/StaticAnalyzer/Frontend/enable.gni")
static_library("plugin") { static_library("plugin") {
output_name = "clangTidyPlugin" output_name = "clangTidyPlugin"
configs += [ "//llvm/utils/gn/build:clang_code" ] configs += [ "//llvm/utils/gn/build:clang_code" ]
deps = [ deps = [
"//clang-tools-extra/clang-tidy", "//clang-tools-extra/clang-tidy",
"//clang-tools-extra/clang-tidy/abseil", "//clang-tools-extra/clang-tidy/abseil",
"//clang-tools-extra/clang-tidy/android", "//clang-tools-extra/clang-tidy/android",
"//clang-tools-extra/clang-tidy/boost", "//clang-tools-extra/clang-tidy/boost",
"//clang-tools-extra/clang-tidy/bugprone", "//clang-tools-extra/clang-tidy/bugprone",
"//clang-tools-extra/clang-tidy/cert", "//clang-tools-extra/clang-tidy/cert",
"//clang-tools-extra/clang-tidy/cppcoreguidelines", "//clang-tools-extra/clang-tidy/cppcoreguidelines",
"//clang-tools-extra/clang-tidy/fuchsia", "//clang-tools-extra/clang-tidy/fuchsia",
"//clang-tools-extra/clang-tidy/google", "//clang-tools-extra/clang-tidy/google",
"//clang-tools-extra/clang-tidy/hicpp", "//clang-tools-extra/clang-tidy/hicpp",
"//clang-tools-extra/clang-tidy/linuxkernel",
"//clang-tools-extra/clang-tidy/llvm", "//clang-tools-extra/clang-tidy/llvm",
"//clang-tools-extra/clang-tidy/misc", "//clang-tools-extra/clang-tidy/misc",
"//clang-tools-extra/clang-tidy/modernize", "//clang-tools-extra/clang-tidy/modernize",
"//clang-tools-extra/clang-tidy/objc", "//clang-tools-extra/clang-tidy/objc",
"//clang-tools-extra/clang-tidy/openmp",
"//clang-tools-extra/clang-tidy/performance", "//clang-tools-extra/clang-tidy/performance",
"//clang-tools-extra/clang-tidy/portability", "//clang-tools-extra/clang-tidy/portability",
"//clang-tools-extra/clang-tidy/readability", "//clang-tools-extra/clang-tidy/readability",
"//clang-tools-extra/clang-tidy/zircon", "//clang-tools-extra/clang-tidy/zircon",
"//clang/lib/AST", "//clang/lib/AST",
"//clang/lib/ASTMatchers", "//clang/lib/ASTMatchers",
"//clang/lib/Basic", "//clang/lib/Basic",
"//clang/lib/Frontend", "//clang/lib/Frontend",
Show All 12 Lines

llvm/trunk/utils/gn/secondary/clang/tools/libclang/BUILD.gn

Show All 33 Lines deps = [
"//llvm/lib/IR", "//llvm/lib/IR",
"//llvm/lib/Support", "//llvm/lib/Support",
"//llvm/lib/Target:TargetsToBuild", "//llvm/lib/Target:TargetsToBuild",
] ]
if (clang_enable_arcmt) { if (clang_enable_arcmt) {
deps += [ "//clang/lib/ARCMigrate" ] deps += [ "//clang/lib/ARCMigrate" ]
} }
defines = []
# FIXME: Once the GN build has a way to select which bits to build,
# only include this dependency if clang-tools-extra is part of the build.
# FIXME: libclang depending on anything in clang-tools-extra seems like
# a layering violation.
if (true) {
defines += [ "CLANG_TOOL_EXTRA_BUILD" ]
deps += [
"//clang-tools-extra/clang-include-fixer/plugin",
"//clang-tools-extra/clang-tidy/plugin",
]
}
if (host_os == "win") { if (host_os == "win") {
defines = [ "_CINDEX_LIB_" ] defines += [ "_CINDEX_LIB_" ]
} }
sources = [ sources = [
"../../include/clang-c/Index.h", "../../include/clang-c/Index.h",
"ARCMigrate.cpp", "ARCMigrate.cpp",
"BuildSystem.cpp", "BuildSystem.cpp",
"CIndex.cpp", "CIndex.cpp",
"CIndexCXX.cpp", "CIndexCXX.cpp",
Show All 39 Lines