[PATCH] D83551: [PATCH 2/4][Sema][AArch64] Add semantics for arm_sve_vector_bits attribute
Aaron Ballman via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Mon Jul 20 08:58:36 PDT 2020
aaron.ballman added a comment.
I like the way the code is shaping up with the slightly altered design, nice!
================
Comment at: clang/lib/AST/ASTContext.cpp:1872
+unsigned getSveVectorWidth(const Type *T) {
+ // Get the vector size from the 'arm_sve_vector_bits' attribute via the
----------------
Should declare the method as `static`.
================
Comment at: clang/lib/AST/ASTContext.cpp:1887
+
+unsigned getSvePredWidth(const Type *T) { return getSveVectorWidth(T) / 8; }
+
----------------
Should this be dividing by the number of bits in a char for the target as opposed to hard-coding to 8?
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D83551/new/
https://reviews.llvm.org/D83551
More information about the cfe-commits
mailing list