[llvm] [clang-tools-extra] [clang] [Clang][AArch64] Add fix vector types to header into SVE (PR #73258)

Paul Walker via cfe-commits cfe-commits at lists.llvm.org
Fri Dec 8 06:36:44 PST 2023


================
@@ -2355,13 +2357,7 @@ void NeonEmitter::run(raw_ostream &OS) {
 
   OS << "#include <arm_bf16.h>\n";
 
-  // Emit NEON-specific scalar typedefs.
-  OS << "typedef float float32_t;\n";
-  OS << "typedef __fp16 float16_t;\n";
-
-  OS << "#ifdef __aarch64__\n";
-  OS << "typedef double float64_t;\n";
-  OS << "#endif\n\n";
+  OS << "#include <arm_vector_types.h>\n";
----------------
paulwalker-arm wrote:

I guess there's a question as to why the poly types have been omitted.  We don't need to be that precious about the header containing only the bare minimum types that are needed across NEON and SVE.  I've seen circumstances where users have wanted the types but nothing else.  I know this is not the goal of this patch but it's a step in that direction.

https://github.com/llvm/llvm-project/pull/73258


More information about the cfe-commits mailing list