================
@@ -1391,7 +1391,8 @@ void ASTContext::InitBuiltinTypes(const TargetInfo
&Target,
}
if (Target.hasAArch64SVETypes() ||
- (AuxTarget && AuxTarget->hasAArch64SVETypes())) {
+ (AuxTarget && AuxTarget->hasAArch64SVETypes()) ||
+ Target.hasArmMFloat8Type()) {
----------------
paulwalker-arm wrote:
This might be a bum steer but I'm wondering if `Target.hasArmMFloat8Type()` is
needed here. If you ignore the name I think `hasAArch64SVETypes()` effectively
means "are_the_types_defined_in_AArch64SVEACLETypes.def_available?".
If you agree that the new types should not be hidden behind a feature flag then
perhaps the original code just works without alteration.
https://github.com/llvm/llvm-project/pull/97277
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
