[PATCH] D97240: [AArch64][GlobalISel] Fix manual selection for v4s16 and v8s8 G_DUP
Amara Emerson via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Feb 23 22:25:11 PST 2021
aemerson accepted this revision.
aemerson added inline comments.
This revision is now accepted and ready to land.
================
Comment at: llvm/lib/Target/AArch64/GISel/AArch64InstructionSelector.cpp:3209
+ return false;
+ } else if (SrcTy.getSizeInBits() == 8) {
+ if (NumElts == 16)
----------------
I think it's probably more readable if you just do a sequence of checks using LLT comparisons. I.e.
```if (SrcTy == LLT::vector(8, 16))
....
else if (SrcTy == LLT::vector(4, 16))
....
etc
```
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D97240/new/
https://reviews.llvm.org/D97240
More information about the llvm-commits
mailing list