[compiler-rt] [compiler-rt][AArch64] Rewrite SME routines to all use __aarch64_cpu_features. (PR #119414)

via llvm-commits llvm-commits at lists.llvm.org
Tue Dec 10 08:57:43 PST 2024


github-actions[bot] wrote:

<!--LLVM CODE FORMAT COMMENT: {clang-format}-->


:warning: C/C++ code formatter, clang-format found issues in your code. :warning:

<details>
<summary>
You can test this locally with the following command:
</summary>

``````````bash
git-clang-format --diff 5a0d73b1da880e475e8187e8962b574174bdd0a9 8db75c0e4784758be2cbe29056b2a3c93a49f6b3 --extensions c,inc -- compiler-rt/lib/builtins/aarch64/sme-abi-assert.c compiler-rt/lib/builtins/cpu_model/aarch64/fmv/baremetal_sme.inc compiler-rt/lib/builtins/cpu_model/aarch64.c
``````````

</details>

<details>
<summary>
View the diff from clang-format here.
</summary>

``````````diff
diff --git a/compiler-rt/lib/builtins/aarch64/sme-abi-assert.c b/compiler-rt/lib/builtins/aarch64/sme-abi-assert.c
index 9df2a20462..4333353f8d 100644
--- a/compiler-rt/lib/builtins/aarch64/sme-abi-assert.c
+++ b/compiler-rt/lib/builtins/aarch64/sme-abi-assert.c
@@ -6,5 +6,5 @@
 // which features are set at runtime.
 
 #include "../cpu_model/AArch64CPUFeatures.inc"
-_Static_assert(FEAT_SVE== 30, "sme-abi.S assumes FEAT_SVE = 30");
-_Static_assert(FEAT_SME== 42, "sme-abi.S assumes FEAT_SME = 42");
+_Static_assert(FEAT_SVE == 30, "sme-abi.S assumes FEAT_SVE = 30");
+_Static_assert(FEAT_SME == 42, "sme-abi.S assumes FEAT_SME = 42");
diff --git a/compiler-rt/lib/builtins/cpu_model/aarch64/fmv/baremetal_sme.inc b/compiler-rt/lib/builtins/cpu_model/aarch64/fmv/baremetal_sme.inc
index e01f848f41..f188e84808 100644
--- a/compiler-rt/lib/builtins/cpu_model/aarch64/fmv/baremetal_sme.inc
+++ b/compiler-rt/lib/builtins/cpu_model/aarch64/fmv/baremetal_sme.inc
@@ -4,9 +4,10 @@
 #if defined(COMPILER_RT_SHARED_LIB)
 __attribute__((weak))
 #endif
-extern _Bool __aarch64_sme_accessible(void);
+extern _Bool
+__aarch64_sme_accessible(void);
 
-static _Bool has_sme(void)  {
+static _Bool has_sme(void) {
 #if defined(COMPILER_RT_SHARED_LIB)
   if (!__aarch64_sme_accessible)
     return 0;
@@ -15,7 +16,7 @@ static _Bool has_sme(void)  {
 }
 
 void __init_cpu_features_resolver(unsigned long hwcap,
-                                  const __ifunc_arg_t *arg) { }
+                                  const __ifunc_arg_t *arg) {}
 
 void CONSTRUCTOR_ATTRIBUTE __init_cpu_features(void) {
   // CPU features already initialized.

``````````

</details>


https://github.com/llvm/llvm-project/pull/119414


More information about the llvm-commits mailing list