sdesmalen marked an inline comment as done. sdesmalen added inline comments.
================ Comment at: clang/lib/CodeGen/CGBuiltin.cpp:5292 + { #NameBase, SVE::BI__builtin_sve_##NameBase, 0, 0, TypeModifier } +static const NeonIntrinsicInfo AArch64SVEIntrinsicMap[] = { +#define GET_SVE_LLVM_INTRINSIC_MAP ---------------- SjoerdMeijer wrote: > I am wondering if it is confusing/correct to use NeonInstrinsicInfo here? We can reuse the same info-struct and find function, but the names are indeed misleading. I've renamed these. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D75470/new/ https://reviews.llvm.org/D75470 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits