[PATCH] D119926: [Clang][AArch64] Enable _Float16 _Complex type

Paul Walker via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Feb 16 02:53:31 PST 2022


paulwalker-arm added inline comments.


================
Comment at: clang/include/clang/AST/ASTContext.h:1117
   CanQualType BFloat16Ty;
-  CanQualType Float16Ty; // C11 extension ISO/IEC TS 18661-3
+  CanQualType Float16Ty, Float16ComplexTy; // C11 extension ISO/IEC TS 18661-3
   CanQualType VoidPtrTy, NullPtrTy;
----------------
Is this necessary? I don't see any other `*ComplexTy` types in this file (i.e. there is no `DoubleComplexTy`).


================
Comment at: clang/lib/AST/ASTContext.cpp:1330
   InitBuiltinType(Float16Ty,           BuiltinType::Float16);
+  InitBuiltinType(Float16ComplexTy,    BuiltinType::Float16);
 
----------------
As above.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D119926/new/

https://reviews.llvm.org/D119926



More information about the cfe-commits mailing list