[clang] bc94a9b - Silence some sign comparison warnings; NFC
Aaron Ballman via cfe-commits
cfe-commits at lists.llvm.org
Thu Jan 19 09:27:02 PST 2023
Author: Aaron Ballman
Date: 2023-01-19T12:26:53-05:00
New Revision: bc94a9b2eceba7effa032a5ce48419da2e69c8e9
URL: https://github.com/llvm/llvm-project/commit/bc94a9b2eceba7effa032a5ce48419da2e69c8e9
DIFF: https://github.com/llvm/llvm-project/commit/bc94a9b2eceba7effa032a5ce48419da2e69c8e9.diff
LOG: Silence some sign comparison warnings; NFC
The std::optional implementation in MSVC causes this code to produce a
sign comparison warning. This ensures the types are the same sign.
Added:
Modified:
clang/lib/Basic/Targets/AArch64.cpp
Removed:
################################################################################
diff --git a/clang/lib/Basic/Targets/AArch64.cpp b/clang/lib/Basic/Targets/AArch64.cpp
index 3a6e8f151a2d7..015deeb84524e 100644
--- a/clang/lib/Basic/Targets/AArch64.cpp
+++ b/clang/lib/Basic/Targets/AArch64.cpp
@@ -60,14 +60,14 @@ void AArch64TargetInfo::setArchFeatures() {
HasLSE = true;
HasRDM = true;
} else if (ArchInfo->Version.getMajor() == 8) {
- if (ArchInfo->Version.getMinor() >= 7) {
+ if (ArchInfo->Version.getMinor() >= 7u) {
HasWFxT = true;
}
- if (ArchInfo->Version.getMinor() >= 6) {
+ if (ArchInfo->Version.getMinor() >= 6u) {
HasBFloat16 = true;
HasMatMul = true;
}
- if (ArchInfo->Version.getMinor() >= 5) {
+ if (ArchInfo->Version.getMinor() >= 5u) {
HasAlternativeNZCV = true;
HasFRInt3264 = true;
HasSSBS = true;
@@ -75,28 +75,28 @@ void AArch64TargetInfo::setArchFeatures() {
HasPredRes = true;
HasBTI = true;
}
- if (ArchInfo->Version.getMinor() >= 4) {
+ if (ArchInfo->Version.getMinor() >= 4u) {
HasDotProd = true;
HasDIT = true;
HasFlagM = true;
}
- if (ArchInfo->Version.getMinor() >= 3) {
+ if (ArchInfo->Version.getMinor() >= 3u) {
HasRCPC = true;
FPU |= NeonMode;
}
- if (ArchInfo->Version.getMinor() >= 2) {
+ if (ArchInfo->Version.getMinor() >= 2u) {
HasCCPP = true;
}
- if (ArchInfo->Version.getMinor() >= 1) {
+ if (ArchInfo->Version.getMinor() >= 1u) {
HasCRC = true;
HasLSE = true;
HasRDM = true;
}
} else if (ArchInfo->Version.getMajor() == 9) {
- if (ArchInfo->Version.getMinor() >= 2) {
+ if (ArchInfo->Version.getMinor() >= 2u) {
HasWFxT = true;
}
- if (ArchInfo->Version.getMinor() >= 1) {
+ if (ArchInfo->Version.getMinor() >= 1u) {
HasBFloat16 = true;
HasMatMul = true;
}
More information about the cfe-commits
mailing list