[PATCH] D102262: [ValueTypes] Rename MVT::getVectorNumElements() to MVT::getVectorMinNumElements(). Fix some misuses of getVectorNumElements()

Craig Topper via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue May 11 15:58:32 PDT 2021


craig.topper added a comment.

In D102262#2752042 <https://reviews.llvm.org/D102262#2752042>, @sdesmalen wrote:

> 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.

There are failures coming from computeRegisterProperties on multiple targets that cause a very high test fallout.


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