[PATCH] D57310: GlobalISel: Consolidate load/store legalization

Matt Arsenault via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Jan 27 15:38:57 PST 2019


arsenm created this revision.
arsenm added reviewers: aemerson, aditya_nandakumar, paquette, volkan, dsanders.
Herald added subscribers: jfb, javed.absar, kristof.beyls, rovka, nhaehnle, wdng, jvesely.

The fewerElementsVectors implementation for load/stores
handles the scalar reduction case just as well, so drop
the redundant code in narrowScalar. This also introduces
support for narrowing irregular size breakdowns for scalars.


https://reviews.llvm.org/D57310

Files:
  include/llvm/CodeGen/GlobalISel/LegalizerHelper.h
  lib/CodeGen/GlobalISel/LegalizerHelper.cpp
  lib/Target/AMDGPU/AMDGPULegalizerInfo.cpp
  test/CodeGen/AArch64/GlobalISel/arm64-fallback.ll
  test/CodeGen/AMDGPU/GlobalISel/legalize-load.mir
  test/CodeGen/AMDGPU/GlobalISel/legalize-store.mir
  test/CodeGen/X86/GlobalISel/x86_64-fallback.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D57310.183781.patch
Type: text/x-patch
Size: 23510 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20190127/3f2e559a/attachment.bin>


More information about the llvm-commits mailing list