[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