[all-commits] [llvm/llvm-project] 8a52fe: [AMDGPU] SILoadStoreOptimizer: tweak API of Combin...

Jay Foad via All-commits all-commits at lists.llvm.org
Thu Jan 27 10:24:05 PST 2022


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 8a52fef1e06c0b8fcf7fede9ec1c5fc1cf326be8
      https://github.com/llvm/llvm-project/commit/8a52fef1e06c0b8fcf7fede9ec1c5fc1cf326be8
  Author: Jay Foad <jay.foad at amd.com>
  Date:   2022-01-27 (Thu, 27 Jan 2022)

  Changed paths:
    M llvm/lib/Target/AMDGPU/SILoadStoreOptimizer.cpp

  Log Message:
  -----------
  [AMDGPU] SILoadStoreOptimizer: tweak API of CombineInfo::setMI. NFC.

Change CombineInfo::setMI to take a reference to the
SILoadStoreOptimizer instance, for easy access to common fields like
TII and STM.

Differential Revision: https://reviews.llvm.org/D118366


  Commit: 94a4594c5483a47b0dd17957093793dab610aff7
      https://github.com/llvm/llvm-project/commit/94a4594c5483a47b0dd17957093793dab610aff7
  Author: Jay Foad <jay.foad at amd.com>
  Date:   2022-01-27 (Thu, 27 Jan 2022)

  Changed paths:
    M llvm/lib/Target/AMDGPU/SILoadStoreOptimizer.cpp

  Log Message:
  -----------
  [AMDGPU] SILoadStoreOptimizer: use separate lists for AGPR instructions

Using separate lists for AGPR and non-AGPR instructions seems like a
cleaner solution than putting them all in the same list and then later
refusing to merge instructions of different AGPR-ness.

Differential Revision: https://reviews.llvm.org/D118367


  Commit: 4b133cee8026cb304ce486f090be0a5ec6aa4a27
      https://github.com/llvm/llvm-project/commit/4b133cee8026cb304ce486f090be0a5ec6aa4a27
  Author: Jay Foad <jay.foad at amd.com>
  Date:   2022-01-27 (Thu, 27 Jan 2022)

  Changed paths:
    M llvm/lib/Target/AMDGPU/SILoadStoreOptimizer.cpp

  Log Message:
  -----------
  [AMDGPU] SILoadStoreOptimizer: reject AGPR DS_WRITE sooner

Rejecting AGPR DS_WRITE instructions before adding them to any mergeable
list seems cleaner than adding them to the list and rejecting them
later.

Differential Revision: https://reviews.llvm.org/D118368


Compare: https://github.com/llvm/llvm-project/compare/c9baa5608bad...4b133cee8026


More information about the All-commits mailing list