[llvm] [lldb][bazel] Remove stale Bazel deps (PR #228580)
via llvm-commits
llvm-commits at lists.llvm.org
Fri Oct 2 13:51:08 PDT 2026
https://github.com/lexi-nadia updated https://github.com/llvm/llvm-project/pull/228580
>From 16e808fddd25d966d3878a3197c957e4aa5f9e0b Mon Sep 17 00:00:00 2001
From: Lexi Bromfield <lexinadia at google.com>
Date: Thu, 1 Oct 2026 22:59:25 +0000
Subject: [PATCH] [lldb][bazel] Remove stale Bazel deps
- Core's dependency on PluginCPlusPlusLanguageHeaders and
PluginObjCLanguageHeaders was removed in
897cc3ee429a62e1dfd77a602db96e9884671f93.
- Expression's dependency on PluginSymbolFileDWARFHeaders was removed in
59c5d531991ca6d3cdd7072a13331dd5b0a5b321.
- Interpreter's dependency on API was removed in
ed3d4a78ba8b23ab5d3fb629029695a03818d254.
These edges made LLDB core libraries transitively depend on plugins
(and, through :API, on every default plugin). The build passes with
layering_check after removing them.
---
.../llvm-project-overlay/lldb/BUILD.bazel | 19 ++-----------------
1 file changed, 2 insertions(+), 17 deletions(-)
diff --git a/utils/bazel/llvm-project-overlay/lldb/BUILD.bazel b/utils/bazel/llvm-project-overlay/lldb/BUILD.bazel
index a830dfba92b69..c89aa94d1ea91 100644
--- a/utils/bazel/llvm-project-overlay/lldb/BUILD.bazel
+++ b/utils/bazel/llvm-project-overlay/lldb/BUILD.bazel
@@ -260,11 +260,11 @@ cc_library(
name = "API",
srcs = glob([
"source/API/**/*.cpp",
+ "source/API/**/*.h",
]),
hdrs = glob(["include/lldb/API/**/*.h"]) + [":lldb-sbapi-dwarf-enums"],
includes = ["include"],
deps = [
- ":APIPrivateHeaders",
":Breakpoint",
":Commands",
":Core",
@@ -273,6 +273,7 @@ cc_library(
":Headers",
":Host",
":Initialization",
+ ":Interpreter",
":InterpreterHeaders",
":Symbol",
":SymbolHeaders",
@@ -359,7 +360,6 @@ cc_library(
":SymbolHeaders",
":TargetHeaders",
":Utility",
- "//lldb/source/Plugins:PluginSymbolFileDWARFHeaders",
"//llvm:BinaryFormat",
"//llvm:Core",
"//llvm:DebugInfoDWARF",
@@ -412,23 +412,12 @@ cc_library(
includes = ["include"],
)
-cc_library(
- name = "APIPrivateHeaders",
- hdrs = glob(["source/API/**/*.h"]),
- features = ["-parse_headers"], # Would otherwise have circular dependencies
- includes = ["source"],
- deps = [
- ":APIHeaders",
- ],
-)
-
cc_library(
name = "InterpreterHeaders",
hdrs = glob(["include/lldb/Interpreter/**/*.h"]),
features = ["-parse_headers"], # Would otherwise have circular dependencies
includes = ["include"],
deps = [
- ":APIHeaders",
":Headers",
],
)
@@ -463,8 +452,6 @@ cc_library(
name = "Interpreter",
srcs = glob(["source/Interpreter/**/*.cpp"]),
deps = [
- ":API",
- ":APIPrivateHeaders",
":Commands",
":Core",
":DataFormatters",
@@ -797,8 +784,6 @@ cc_library(
":Utility",
":Version",
"//clang:driver",
- "//lldb/source/Plugins:PluginCPlusPlusLanguageHeaders",
- "//lldb/source/Plugins:PluginObjCLanguageHeaders",
"//llvm:BinaryFormat",
"//llvm:Demangle",
"//llvm:Support",
More information about the llvm-commits
mailing list