This is an archive of the discontinued LLVM Phabricator instance.

Move dependency llvm:AllTargetsAsmParsers from Translation to ExecutionEngine.
ClosedPublic

Authored by jurahul on Nov 23 2021, 1:25 PM.

Reviewers

Commits

Summary

Fixes a minor issue in https://reviews.llvm.org/D114338, which seems incorrectly added the llvm:AllTargetsAsmParsers dependency to Translation in bazel build files.

jurahul requested review of this revision.Nov 23 2021, 1:25 PM

LGTM, thanks

This revision is now accepted and ready to land.Nov 23 2021, 1:46 PM

This revision was automatically updated to reflect the committed changes.

Path

Size

utils/

bazel/

llvm-project-overlay/

mlir/

2 lines

Show First 20 Lines • Show All 4,986 Lines • ▼ Show 20 Lines	cc_library(
]),		]),
hdrs = ["include/mlir/Translation.h"],		hdrs = ["include/mlir/Translation.h"],
includes = ["include"],		includes = ["include"],
deps = [		deps = [
":Analysis",		":Analysis",
":IR",		":IR",
":Parser",		":Parser",
":Support",		":Support",
"//llvm:AllTargetsAsmParsers",
"//llvm:Support",		"//llvm:Support",
],		],
)		)

cc_library(		cc_library(
name = "ToLLVMIRTranslation",		name = "ToLLVMIRTranslation",
srcs = [		srcs = [
"lib/Target/LLVMIR/DebugTranslation.cpp",		"lib/Target/LLVMIR/DebugTranslation.cpp",
▲ Show 20 Lines • Show All 236 Lines • ▼ Show 20 Lines	cc_library(
includes = ["include"],		includes = ["include"],
deps = [		deps = [
":AllToLLVMIRTranslations",		":AllToLLVMIRTranslations",
":IR",		":IR",
":LLVMDialect",		":LLVMDialect",
":Support",		":Support",
":ToLLVMIRTranslation",		":ToLLVMIRTranslation",
":Translation",		":Translation",
		"//llvm:AllTargetsAsmParsers",
"//llvm:BitReader",		"//llvm:BitReader",
"//llvm:BitWriter",		"//llvm:BitWriter",
"//llvm:Core",		"//llvm:Core",
"//llvm:ExecutionEngine",		"//llvm:ExecutionEngine",
"//llvm:MC",		"//llvm:MC",
"//llvm:OrcJIT",		"//llvm:OrcJIT",
"//llvm:Support",		"//llvm:Support",
"//llvm:Target", # fixdeps: keep		"//llvm:Target", # fixdeps: keep
▲ Show 20 Lines • Show All 2,388 Lines • Show Last 20 Lines