[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