[PATCH] D79480: [SveEmitter] Add builtins for SVE2 Polynomial arithmetic

Eli Friedman via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed May 6 09:07:35 PDT 2020


efriedma accepted this revision.
efriedma added a comment.
This revision is now accepted and ready to land.

LGTM with one minor comment



================
Comment at: clang/lib/CodeGen/CGBuiltin.cpp:7813
+  llvm::ScalableVectorType *Ty = getSVEType(TypeFlags);
+  return Builder.CreateBitCast(Call, Ty);
+}
----------------
Please add a FIXME for big-endian. :)


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D79480/new/

https://reviews.llvm.org/D79480





More information about the cfe-commits mailing list