[llvm] [bazel] minor formatting fixes (PR #122964)
Jorge Gorbe Moya via llvm-commits
llvm-commits at lists.llvm.org
Tue Jan 14 12:31:51 PST 2025
https://github.com/slackito created https://github.com/llvm/llvm-project/pull/122964
Added `# keep sorted` to a couple of long-ish lists of files that buildifier didn't automatically sort by default.
Changed a couple of one-element `toolchains` attributes to the single-line format.
>From e458a4eb91c6b0c943bd09788dd4a45e34cd485d Mon Sep 17 00:00:00 2001
From: Jorge Gorbe Moya <jgorbe at google.com>
Date: Tue, 14 Jan 2025 12:29:26 -0800
Subject: [PATCH] [bazel] minor formatting fixes
Added `# keep sorted` to a couple of long-ish lists of files that
buildifier didn't automatically sort by default.
Changed a couple of one-element `toolchains` attributes to the
single-line format.
---
.../llvm-project-overlay/clang/BUILD.bazel | 58 +++++++++----------
1 file changed, 27 insertions(+), 31 deletions(-)
diff --git a/utils/bazel/llvm-project-overlay/clang/BUILD.bazel b/utils/bazel/llvm-project-overlay/clang/BUILD.bazel
index 2ad79ce95b1670..ab9d24c111ce32 100644
--- a/utils/bazel/llvm-project-overlay/clang/BUILD.bazel
+++ b/utils/bazel/llvm-project-overlay/clang/BUILD.bazel
@@ -693,32 +693,33 @@ cc_library(
],
includes = ["include"],
textual_hdrs = [
- "include/clang/Basic/arm_fp16.inc",
- "include/clang/Basic/arm_mve_builtins.inc",
- "include/clang/Basic/arm_immcheck_types.inc",
- "include/clang/Basic/arm_mve_builtin_aliases.inc",
- "include/clang/Basic/arm_mve_builtin_cg.inc",
- "include/clang/Basic/arm_mve_builtin_sema.inc",
- "include/clang/Basic/arm_sme_builtins.inc",
- "include/clang/Basic/arm_sme_builtin_cg.inc",
- "include/clang/Basic/arm_neon.inc",
+ # keep sorted
"include/clang/Basic/AttrHasAttributeImpl.inc",
"include/clang/Basic/AttrList.inc",
"include/clang/Basic/AttrSubMatchRulesList.inc",
"include/clang/Basic/DiagnosticASTKinds.inc",
- "include/clang/Basic/DiagnosticGroups.inc",
- "include/clang/Basic/DiagnosticRefactoringKinds.inc",
"include/clang/Basic/DiagnosticAnalysisKinds.inc",
- "include/clang/Basic/DiagnosticSemaKinds.inc",
"include/clang/Basic/DiagnosticCommentKinds.inc",
- "include/clang/Basic/DiagnosticParseKinds.inc",
- "include/clang/Basic/DiagnosticLexKinds.inc",
- "include/clang/Basic/DiagnosticSerializationKinds.inc",
- "include/clang/Basic/DiagnosticFrontendKinds.inc",
- "include/clang/Basic/DiagnosticDriverKinds.inc",
- "include/clang/Basic/DiagnosticCrossTUKinds.inc",
"include/clang/Basic/DiagnosticCommonKinds.inc",
+ "include/clang/Basic/DiagnosticCrossTUKinds.inc",
+ "include/clang/Basic/DiagnosticDriverKinds.inc",
+ "include/clang/Basic/DiagnosticFrontendKinds.inc",
+ "include/clang/Basic/DiagnosticGroups.inc",
"include/clang/Basic/DiagnosticIndexName.inc",
+ "include/clang/Basic/DiagnosticLexKinds.inc",
+ "include/clang/Basic/DiagnosticParseKinds.inc",
+ "include/clang/Basic/DiagnosticRefactoringKinds.inc",
+ "include/clang/Basic/DiagnosticSemaKinds.inc",
+ "include/clang/Basic/DiagnosticSerializationKinds.inc",
+ "include/clang/Basic/arm_fp16.inc",
+ "include/clang/Basic/arm_immcheck_types.inc",
+ "include/clang/Basic/arm_mve_builtin_aliases.inc",
+ "include/clang/Basic/arm_mve_builtin_cg.inc",
+ "include/clang/Basic/arm_mve_builtin_sema.inc",
+ "include/clang/Basic/arm_mve_builtins.inc",
+ "include/clang/Basic/arm_neon.inc",
+ "include/clang/Basic/arm_sme_builtin_cg.inc",
+ "include/clang/Basic/arm_sme_builtins.inc",
] + glob([
"include/clang/Basic/*.def",
]),
@@ -1034,9 +1035,7 @@ cc_library(
] + glob([
"include/clang/AST/*.def",
]),
- toolchains = [
- ":workspace_root",
- ],
+ toolchains = [":workspace_root"],
deps = [
":ast_attr_gen",
":ast_bytecode_opcodes_gen",
@@ -1848,22 +1847,23 @@ builtin_headers = glob(
"lib/Headers/ppc_wrappers/nmmintrin.h",
],
) + [
+ # keep sorted
+ "lib/Headers/arm_bf16.h",
"lib/Headers/arm_cde.h",
"lib/Headers/arm_fp16.h",
"lib/Headers/arm_mve.h",
"lib/Headers/arm_neon.h",
- "lib/Headers/arm_sve.h",
"lib/Headers/arm_sme.h",
+ "lib/Headers/arm_sve.h",
"lib/Headers/arm_vector_types.h",
- "lib/Headers/arm_bf16.h",
- "lib/Headers/module.modulemap",
- "lib/Headers/riscv_vector.h",
"lib/Headers/cuda_wrappers/algorithm",
"lib/Headers/cuda_wrappers/complex",
"lib/Headers/cuda_wrappers/new",
+ "lib/Headers/module.modulemap",
"lib/Headers/openmp_wrappers/cmath",
"lib/Headers/openmp_wrappers/complex",
"lib/Headers/openmp_wrappers/new",
+ "lib/Headers/riscv_vector.h",
]
genrule(
@@ -1878,9 +1878,7 @@ genrule(
cp $$src $$target
done""",
output_to_bindir = 1,
- toolchains = [
- ":workspace_root",
- ],
+ toolchains = [":workspace_root"],
)
cc_library(
@@ -2606,9 +2604,7 @@ cc_binary(
"-I$(GENDIR)/$(WORKSPACE_ROOT)/clang/tools/clang-linker-wrapper",
],
stamp = 0,
- toolchains = [
- ":workspace_root",
- ],
+ toolchains = [":workspace_root"],
deps = [
":basic",
":linker_wrapper_opts_gen",
More information about the llvm-commits
mailing list