[PATCH] D23776: AMDGPU/SI: Canonicalize offset order for merged DS instructions

Tom Stellard via llvm-commits llvm-commits at lists.llvm.org
Mon Aug 22 08:40:40 PDT 2016


tstellarAMD created this revision.
tstellarAMD added a reviewer: arsenm.
tstellarAMD added a subscriber: llvm-commits.
Herald added subscribers: kzhuravl, arsenm.

If the scheduler clusters the loads, then the offsets will be sorted,
but it is possible for the scheduler to scheduler loads together
without out explicitly clustering them, which would give us non-sorted
offsets.

Also, we will want to do this if we move the load/store optimizer before
the scheduler.

https://reviews.llvm.org/D23776

Files:
  lib/Target/AMDGPU/SILoadStoreOptimizer.cpp
  test/CodeGen/AMDGPU/ds_read2_offset_order.ll
  test/CodeGen/AMDGPU/ds_read2st64.ll
  test/CodeGen/AMDGPU/load-local-i16.ll
  test/CodeGen/AMDGPU/load-local-i32.ll
  test/CodeGen/AMDGPU/load-local-i8.ll
  test/CodeGen/AMDGPU/local-64.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D23776.68866.patch
Type: text/x-patch
Size: 10637 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20160822/865845c5/attachment.bin>


More information about the llvm-commits mailing list