[PATCH] D80147: AMDGPU/GlobalISel: Fix splitting 64-bit extensions

[Matt Arsenault via Phabricator via llvm-commits]

arsenm marked 2 inline comments as done.
arsenm added inline comments.


================
Comment at: llvm/lib/Target/AMDGPU/AMDGPURegisterBankInfo.cpp:2308-2310
+        // TODO: For sign extend case, we may be better off emitting another
+        // select rather than a shift of the sign bit (although the shift has
+        // better code size).
----------------
foad wrote:
> Surely the src is always i1 here, so the high part of dst will always be 0 (or undef for anyext)?
Not if it’s G_SEXT


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

https://reviews.llvm.org/D80147