[PATCH] D57029: GlobalISel: Fix narrowScalar for load/store with different mem size

[Matt Arsenault via Phabricator via llvm-commits]

arsenm created this revision.
arsenm added reviewers: dsanders, volkan, aditya_nandakumar, aemerson.
Herald added subscribers: kristof.beyls, rovka, nhaehnle, wdng, jvesely.
arsenm added a parent revision: D55814: GlobalISel: Support narrowing zextload/sextload.

This was ignoring the memory size, and producing multiple loads/stores
if the operand size was different from the memory size.

      

I assume this is the intent of not having an explicit G_ANYEXTLOAD
(although I think that would probably be better).


https://reviews.llvm.org/D57029

Files:
  lib/CodeGen/GlobalISel/LegalizerHelper.cpp
  lib/Target/AMDGPU/AMDGPULegalizerInfo.cpp
  test/CodeGen/AMDGPU/GlobalISel/legalize-load.mir
  test/CodeGen/AMDGPU/GlobalISel/legalize-store.mir

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D57029.182811.patch
Type: text/x-patch
Size: 9117 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20190121/6908b095/attachment-0001.bin>