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

Sander de Smalen via cfe-commits cfe-commits at lists.llvm.org
Mon Nov 27 03:44:45 PST 2023


================
@@ -1280,6 +1280,7 @@ void SVEEmitter::createHeader(raw_ostream &OS) {
   OS << "typedef __SVBfloat16_t svbfloat16_t;\n";
 
   OS << "#include <arm_bf16.h>\n";
+  OS << "#include <arm_vector_type.h>\n";
----------------
sdesmalen-arm wrote:

I'm missing similar changes for NeonEmitter.cpp where this include would be added for arm_neon.h and where the types are removed from arm_neon.h so that we don't end up with duplicate typedefs.

Duplicate typedefs is not an issue when they are exactly the same (from C11 onward), but as I pointed out above, they are subtly different, which leads to issues.

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


More information about the cfe-commits mailing list