[clang] [Clang][AArch64] Fix typo with colon-separated syntax for system registers (PR #105608)
via cfe-commits
cfe-commits at lists.llvm.org
Wed Aug 21 20:23:19 PDT 2024
llvmbot wrote:
<!--LLVM PR SUMMARY COMMENT-->
@llvm/pr-subscribers-backend-aarch64
@llvm/pr-subscribers-backend-arm
Author: None (v01dXYZ)
<details>
<summary>Changes</summary>
The range for Op0 was set to 1 instead of 3.
The description of e493f177eeee84a9c6000ca7c92499233490f1d1 visually explains the encoding of implementation-defined system registers.
Gobolt: https://godbolt.org/z/WK9PqPvGE (comment out the last one to convince yourself they represent the same system register).
---
Full diff: https://github.com/llvm/llvm-project/pull/105608.diff
2 Files Affected:
- (modified) clang/lib/Sema/SemaARM.cpp (+4-4)
- (modified) clang/test/Sema/aarch64-special-register.c (+7)
``````````diff
diff --git a/clang/lib/Sema/SemaARM.cpp b/clang/lib/Sema/SemaARM.cpp
index d8dd4fe16e3af0..cb53c61aa857d7 100644
--- a/clang/lib/Sema/SemaARM.cpp
+++ b/clang/lib/Sema/SemaARM.cpp
@@ -248,16 +248,16 @@ bool SemaARM::BuiltinARMSpecialReg(unsigned BuiltinID, CallExpr *TheCall,
}
}
- SmallVector<int, 5> Ranges;
+ SmallVector<int, 5> FieldBitWidths;
if (FiveFields)
- Ranges.append({IsAArch64Builtin ? 1 : 15, 7, 15, 15, 7});
+ FieldBitWidths.append({IsAArch64Builtin ? 2 : 4, 3, 4, 4, 3});
else
- Ranges.append({15, 7, 15});
+ FieldBitWidths.append({4, 3, 4});
for (unsigned i = 0; i < Fields.size(); ++i) {
int IntField;
ValidString &= !Fields[i].getAsInteger(10, IntField);
- ValidString &= (IntField >= 0 && IntField <= Ranges[i]);
+ ValidString &= (IntField >= 0 && IntField < (1 << FieldBitWidths[i]));
}
if (!ValidString)
diff --git a/clang/test/Sema/aarch64-special-register.c b/clang/test/Sema/aarch64-special-register.c
index 4d2cfd8b37c847..53a2c32b84777c 100644
--- a/clang/test/Sema/aarch64-special-register.c
+++ b/clang/test/Sema/aarch64-special-register.c
@@ -116,6 +116,13 @@ unsigned long rsr64_6(void) {
return __builtin_arm_rsr64("0:1:16:16:2"); //expected-error {{invalid special register for builtin}}
}
+void rsr64_7(unsigned long *r) {
+ // the following three instructions should produce the same assembly
+ r[0] = __builtin_arm_rsr64("ICC_CTLR_EL3");
+ r[1] = __builtin_arm_rsr64("s3_6_c12_c12_4");
+ r[2] = __builtin_arm_rsr64("3:6:12:12:4");
+}
+
__uint128_t rsr128_3(void) {
return __builtin_arm_rsr128("0:1:2"); //expected-error {{invalid special register for builtin}}
}
``````````
</details>
https://github.com/llvm/llvm-project/pull/105608
More information about the cfe-commits
mailing list