================
@@ -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
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to