[PATCH] D149841: AMDGPU/SDAG: Improve {extract,insert}_subvector lowering for 16-bit vectors

[Joe Nash via Phabricator via llvm-commits]

Joe_Nash accepted this revision.
Joe_Nash added inline comments.
This revision is now accepted and ready to land.


================
Comment at: llvm/lib/Target/AMDGPU/AMDGPUISelLowering.cpp:1440
+         (SrcVT == MVT::v4i16 && VT == MVT::v2i16)) &&
+        Start != 1)
+      return Op;
----------------
foad wrote:
> `Start != 1` is redundant now.
I'm not sure the Start == NumElt case will ever be hit.
But it looks like that can be removed in a follow up patch anyway.
LGTM


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D149841