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

Cullen Rhodes via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Aug 18 09:14:09 PDT 2020


c-rhodes accepted this revision.
c-rhodes added a comment.
This revision is now accepted and ready to land.

LGTM, just one minor nit but seems like a nice improvement



================
Comment at: clang/include/clang/AST/ASTContext.h:1318
+  BuiltinVectorTypeInfo
+  getElementTypeForBuiltinVector(const BuiltinType *VecTy) const;
+
----------------
nit: not sure on the name here since it's returning more than the element type, how about `getBuiltinVectorTypeInfo`?


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