================ @@ -8086,23 +8086,21 @@ static void HandleNeonVectorTypeAttr(QualType &CurType, const ParsedAttr &Attr, // Target must have NEON (or MVE, whose vectors are similar enough // not to need a separate attribute) - if (!(S.Context.getTargetInfo().hasFeature("neon") || - S.Context.getTargetInfo().hasFeature("mve") || - S.Context.getTargetInfo().hasFeature("sve") || - S.Context.getTargetInfo().hasFeature("sme") || + if (!(S.Context.getTargetInfo().hasFeature("mve") || IsTargetCUDAAndHostARM) && ---------------- mgabka wrote:
it looks to me like the logic here can be simplified and use of this variable can be removed entirely. After reading https://github.com/llvm/llvm-project/commit/8b0ea48740935d819618d8254fc45d98179b672c It looks to me like somebody already disabled these errors when targeting GPUs, so we could just simplify the logic and emit the errors only when user targets M class CPUs, what do you think? https://github.com/llvm/llvm-project/pull/95224 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits