Author: Daniel Kiss Date: 2022-12-09T16:23:26+01:00 New Revision: 218b77c85057e8aae153edcedfe365061270e70c
URL: https://github.com/llvm/llvm-project/commit/218b77c85057e8aae153edcedfe365061270e70c DIFF: https://github.com/llvm/llvm-project/commit/218b77c85057e8aae153edcedfe365061270e70c.diff LOG: [AArch64][NFC] Move hasFeature fields initiations to the declaration hasFeature fields need to be initialised to false. Easy to miss as missed for hasPAuth and hasFlagM. Maybe the code less error prone like this. Reviewed By: chill Differential Revision: https://reviews.llvm.org/D139622 Added: Modified: clang/lib/Basic/Targets/AArch64.cpp clang/lib/Basic/Targets/AArch64.h Removed: ################################################################################ diff --git a/clang/lib/Basic/Targets/AArch64.cpp b/clang/lib/Basic/Targets/AArch64.cpp index 89ad602b591f5..90a83f62574fc 100644 --- a/clang/lib/Basic/Targets/AArch64.cpp +++ b/clang/lib/Basic/Targets/AArch64.cpp @@ -579,37 +579,6 @@ void AArch64TargetInfo::setFeatureEnabled(llvm::StringMap<bool> &Features, bool AArch64TargetInfo::handleTargetFeatures(std::vector<std::string> &Features, DiagnosticsEngine &Diags) { - FPU = FPUMode; - HasCRC = false; - HasAES = false; - HasSHA2 = false; - HasSHA3 = false; - HasSM4 = false; - HasUnaligned = true; - HasFullFP16 = false; - HasDotProd = false; - HasFP16FML = false; - HasMTE = false; - HasTME = false; - HasLS64 = false; - HasRandGen = false; - HasMatMul = false; - HasBFloat16 = false; - HasSVE2 = false; - HasSVE2AES = false; - HasSVE2SHA3 = false; - HasSVE2SM4 = false; - HasSVE2BitPerm = false; - HasMatmulFP64 = false; - HasMatmulFP32 = false; - HasLSE = false; - HasMOPS = false; - HasD128 = false; - HasRCPC = false; - HasPAuth = false; - - ArchKind = llvm::AArch64::ArchKind::INVALID; - for (const auto &Feature : Features) { if (Feature == "+neon") FPU |= NeonMode; diff --git a/clang/lib/Basic/Targets/AArch64.h b/clang/lib/Basic/Targets/AArch64.h index 1791e462139f3..b971c56ec9cda 100644 --- a/clang/lib/Basic/Targets/AArch64.h +++ b/clang/lib/Basic/Targets/AArch64.h @@ -27,36 +27,36 @@ class LLVM_LIBRARY_VISIBILITY AArch64TargetInfo : public TargetInfo { enum FPUModeEnum { FPUMode, NeonMode = (1 << 0), SveMode = (1 << 1) }; - unsigned FPU; - bool HasCRC; - bool HasAES; - bool HasSHA2; - bool HasSHA3; - bool HasSM4; - bool HasUnaligned; - bool HasFullFP16; - bool HasDotProd; - bool HasFP16FML; - bool HasMTE; - bool HasTME; - bool HasPAuth; - bool HasLS64; - bool HasRandGen; - bool HasMatMul; - bool HasSVE2; - bool HasSVE2AES; - bool HasSVE2SHA3; - bool HasSVE2SM4; - bool HasSVE2BitPerm; - bool HasMatmulFP64; - bool HasMatmulFP32; - bool HasLSE; - bool HasFlagM; - bool HasMOPS; - bool HasD128; - bool HasRCPC; - - llvm::AArch64::ArchKind ArchKind; + unsigned FPU = FPUMode; + bool HasCRC = false; + bool HasAES = false; + bool HasSHA2 = false; + bool HasSHA3 = false; + bool HasSM4 = false; + bool HasUnaligned = true; + bool HasFullFP16 = false; + bool HasDotProd = false; + bool HasFP16FML = false; + bool HasMTE = false; + bool HasTME = false; + bool HasPAuth = false; + bool HasLS64 = false; + bool HasRandGen = false; + bool HasMatMul = false; + bool HasSVE2 = false; + bool HasSVE2AES = false; + bool HasSVE2SHA3 = false; + bool HasSVE2SM4 = false; + bool HasSVE2BitPerm = false; + bool HasMatmulFP64 = false; + bool HasMatmulFP32 = false; + bool HasLSE = false; + bool HasFlagM = false; + bool HasMOPS = false; + bool HasD128 = false; + bool HasRCPC = false; + + llvm::AArch64::ArchKind ArchKind = llvm::AArch64::ArchKind::INVALID; static const Builtin::Info BuiltinInfo[]; _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
