[PATCH] D102262: [ValueTypes] Rename MVT::getVectorNumElements() to MVT::getVectorMinNumElements(). Fix some misuses of getVectorNumElements()
Sander de Smalen via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue May 11 14:00:29 PDT 2021
sdesmalen accepted this revision.
sdesmalen added a comment.
This revision is now accepted and ready to land.
Thanks, this looks like sensible changes to me! Do you have a ballpark figure of how many tests currently fail when adding a check to `getVectorNumElements`? The fact that the RISCV lit-tests already pass sounds promising.
I've taken the liberty to accept the patch, but maybe just wait a day or two to give other reviewers a chance to take a look as well.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D102262/new/
https://reviews.llvm.org/D102262
More information about the llvm-commits
mailing list