Skip to content

[clang][SemaARM][NFC] clang-format aarch64 builtin check #136188

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Apr 17, 2025

Conversation

sarnex
Copy link
Member

@sarnex sarnex commented Apr 17, 2025

Going to modify this code so needs formatting.

@sarnex sarnex marked this pull request as ready for review April 17, 2025 19:48
@sarnex sarnex merged commit 1c35f81 into llvm:main Apr 17, 2025
9 of 13 checks passed
@llvmbot llvmbot added clang Clang issues not falling into any other category backend:ARM backend:AArch64 clang:frontend Language frontend issues, e.g. anything involving "Sema" labels Apr 17, 2025
@llvmbot
Copy link
Member

llvmbot commented Apr 17, 2025

@llvm/pr-subscribers-backend-aarch64

@llvm/pr-subscribers-backend-arm

Author: Nick Sarnie (sarnex)

Changes

Going to modify this code so needs formatting.


Full diff: https://github.com/llvm/llvm-project/pull/136188.diff

1 Files Affected:

  • (modified) clang/lib/Sema/SemaARM.cpp (+4-1)
diff --git a/clang/lib/Sema/SemaARM.cpp b/clang/lib/Sema/SemaARM.cpp
index 5bcbe78e9d633..b5abf244132d8 100644
--- a/clang/lib/Sema/SemaARM.cpp
+++ b/clang/lib/Sema/SemaARM.cpp
@@ -1109,7 +1109,10 @@ bool SemaARM::CheckAArch64BuiltinFunctionCall(const TargetInfo &TI,
   default: return false;
   case AArch64::BI__builtin_arm_dmb:
   case AArch64::BI__builtin_arm_dsb:
-  case AArch64::BI__builtin_arm_isb: l = 0; u = 15; break;
+  case AArch64::BI__builtin_arm_isb:
+    l = 0;
+    u = 15;
+    break;
   case AArch64::BI__builtin_arm_tcancel: l = 0; u = 65535; break;
   }
 

@llvmbot
Copy link
Member

llvmbot commented Apr 17, 2025

@llvm/pr-subscribers-clang

Author: Nick Sarnie (sarnex)

Changes

Going to modify this code so needs formatting.


Full diff: https://github.com/llvm/llvm-project/pull/136188.diff

1 Files Affected:

  • (modified) clang/lib/Sema/SemaARM.cpp (+4-1)
diff --git a/clang/lib/Sema/SemaARM.cpp b/clang/lib/Sema/SemaARM.cpp
index 5bcbe78e9d633..b5abf244132d8 100644
--- a/clang/lib/Sema/SemaARM.cpp
+++ b/clang/lib/Sema/SemaARM.cpp
@@ -1109,7 +1109,10 @@ bool SemaARM::CheckAArch64BuiltinFunctionCall(const TargetInfo &TI,
   default: return false;
   case AArch64::BI__builtin_arm_dmb:
   case AArch64::BI__builtin_arm_dsb:
-  case AArch64::BI__builtin_arm_isb: l = 0; u = 15; break;
+  case AArch64::BI__builtin_arm_isb:
+    l = 0;
+    u = 15;
+    break;
   case AArch64::BI__builtin_arm_tcancel: l = 0; u = 65535; break;
   }
 

IanWood1 pushed a commit to IanWood1/llvm-project that referenced this pull request May 6, 2025
Going to modify this code so needs formatting.

Signed-off-by: Sarnie, Nick <[email protected]>
IanWood1 pushed a commit to IanWood1/llvm-project that referenced this pull request May 6, 2025
Going to modify this code so needs formatting.

Signed-off-by: Sarnie, Nick <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backend:AArch64 backend:ARM clang:frontend Language frontend issues, e.g. anything involving "Sema" clang Clang issues not falling into any other category
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants