[PATCH] D158386: [AArch64][SME] Promote mask for masked load to a similar type size with load value.
Sander de Smalen via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Aug 22 07:07:17 PDT 2023
sdesmalen added inline comments.
================
Comment at: llvm/lib/Target/AArch64/AArch64ISelLowering.cpp:24966
+ // load's type then we have to extend the mask type.
+ if (Load->getExtensionType() != ISD::NON_EXTLOAD &&
+ VT.getVectorElementType().getSizeInBits() >
----------------
dtemirbulatov wrote:
> sdesmalen wrote:
> > Is it necessary to check for the extension type? (i.e. if you remove this condition, do any tests fail?)
> No tests failed, but I think it can only occur if the load is extending.
Do you want to make it into an assert then?
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D158386/new/
https://reviews.llvm.org/D158386
More information about the llvm-commits
mailing list