[PATCH] D75470: [SVE] Auto-generate builtins and header for svld1.

Sander de Smalen via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Mar 10 04:34:05 PDT 2020


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





More information about the cfe-commits mailing list