[PATCH] D105935: WIP: [AArch64][GlobalISel] Legalize bswap <2 x i16>

[Jessica Paquette via Phabricator via llvm-commits]

paquette added inline comments.


================
Comment at: llvm/lib/Target/AArch64/GISel/AArch64InstructionSelector.cpp:3999
       Register InsertReg = MRI.createVirtualRegister(&AArch64::FPR128RegClass);
+      auto SubImm = [&]() {
+        switch (WideTy.getScalarSizeInBits() * NumElts) {
----------------
paquette wrote:
> This looks similar to `getSubRegForClass`? Maybe it's possible to share some code there?
Also, it'd be nice to have a specific test for G_UNMERGE_VALUES selection. I think that this part + that test could be moved into an independent patch.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D105935/new/

https://reviews.llvm.org/D105935