[PATCH] D86100: [Clang][SVE] NFC: Move info about ACLE types into separate function.

Sander de Smalen via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Aug 19 03:06:24 PDT 2020


sdesmalen added inline comments.


================
Comment at: clang/include/clang/AST/ASTContext.h:1318
+  BuiltinVectorTypeInfo
+  getElementTypeForBuiltinVector(const BuiltinType *VecTy) const;
+
----------------
c-rhodes wrote:
> nit: not sure on the name here since it's returning more than the element type, how about `getBuiltinVectorTypeInfo`?
Thanks, you're right that was a misnomer. I've taken your suggestion (and also updated the comment).


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D86100



More information about the cfe-commits mailing list