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; } _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits