[llvm] [bazel] Fix layering_check with macOS targets (PR #91260)
Keith Smiley via llvm-commits
llvm-commits at lists.llvm.org
Mon May 6 12:38:50 PDT 2024
https://github.com/keith created https://github.com/llvm/llvm-project/pull/91260
Upstream in the apple_support repo I've enabled layering checks for macOS builds. These targets violated that since they previously weren't validated.
>From 0c3fcb01a614009df9759ae137e954f58b66cba5 Mon Sep 17 00:00:00 2001
From: Keith Smiley <keithbsmiley at gmail.com>
Date: Mon, 6 May 2024 12:35:31 -0700
Subject: [PATCH] [bazel] Fix layering_check with macOS targets
Upstream in the apple_support repo I've enabled layering checks for
macOS builds. These targets violated that since they previously weren't
validated.
---
.../bazel/llvm-project-overlay/lldb/BUILD.bazel | 15 +++++++++++++--
.../lldb/source/Plugins/BUILD.bazel | 17 +++++++++++++++--
2 files changed, 28 insertions(+), 4 deletions(-)
diff --git a/utils/bazel/llvm-project-overlay/lldb/BUILD.bazel b/utils/bazel/llvm-project-overlay/lldb/BUILD.bazel
index c06d0c1cfe4600..8fb90e850f0005 100644
--- a/utils/bazel/llvm-project-overlay/lldb/BUILD.bazel
+++ b/utils/bazel/llvm-project-overlay/lldb/BUILD.bazel
@@ -494,8 +494,12 @@ objc_library(
"//conditions:default": ["@platforms//:incompatible"],
}),
deps = [
+ ":Headers",
":HostMacOSXHeaders",
":HostMacOSXPrivateHeaders",
+ ":Utility",
+ "//llvm:Support",
+ "//llvm:TargetParser",
],
)
@@ -561,7 +565,10 @@ cc_library(
"//llvm:TargetParser",
"//llvm:config",
] + select({
- "@platforms//os:macos": [":HostMacOSXObjCXX"],
+ "@platforms//os:macos": [
+ ":HostMacOSXObjCXX",
+ ":HostMacOSXPrivateHeaders",
+ ],
"//conditions:default": [],
}),
)
@@ -873,7 +880,11 @@ cc_binary(
"@platforms//os:macos": [],
"//conditions:default": ["@platforms//:incompatible"],
}),
- deps = [":DebugServerCommon"],
+ deps = [
+ ":DebugServerCommon",
+ ":DebugServerCommonHeaders",
+ ":DebugServerCommonMacOSXHeaders",
+ ],
)
cc_binary(
diff --git a/utils/bazel/llvm-project-overlay/lldb/source/Plugins/BUILD.bazel b/utils/bazel/llvm-project-overlay/lldb/source/Plugins/BUILD.bazel
index d705af9167d812..a9a74ae09b3b8b 100644
--- a/utils/bazel/llvm-project-overlay/lldb/source/Plugins/BUILD.bazel
+++ b/utils/bazel/llvm-project-overlay/lldb/source/Plugins/BUILD.bazel
@@ -250,7 +250,12 @@ objc_library(
"@platforms//os:macos": [],
"//conditions:default": ["@platforms//:incompatible"],
}),
- deps = [":PluginPlatformMacOSXObjCXXHeaders"],
+ deps = [
+ ":PluginPlatformMacOSXObjCXXHeaders",
+ "//lldb:Host",
+ "//lldb:HostMacOSXPrivateHeaders",
+ "//llvm:Support",
+ ],
)
cc_library(
@@ -275,6 +280,7 @@ cc_library(
"//lldb:Core",
"//lldb:Headers",
"//lldb:Host",
+ "//lldb:HostMacOSXPrivateHeaders",
"//lldb:InterpreterHeaders",
"//lldb:SymbolHeaders",
"//lldb:TargetHeaders",
@@ -282,7 +288,10 @@ cc_library(
"//llvm:Support",
"//llvm:TargetParser",
] + select({
- "@platforms//os:macos": [":PluginPlatformMacOSXObjCXX"],
+ "@platforms//os:macos": [
+ ":PluginPlatformMacOSXObjCXX",
+ ":PluginPlatformMacOSXObjCXXHeaders",
+ ],
"//conditions:default": [],
}),
)
@@ -1748,6 +1757,10 @@ cc_library(
"//lldb:Host",
"//lldb:HostMacOSXPrivateHeaders",
"//lldb:Symbol",
+ "//lldb:SymbolHeaders",
+ "//lldb:TargetHeaders",
+ "//lldb:Utility",
+ "//llvm:Support",
],
)
More information about the llvm-commits
mailing list