[PATCH] D98940: [AMDGPU] Allow index optimisation in SIPreEmitPeephole for bundles

Tony Tye via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Mar 19 07:07:46 PDT 2021


t-tye added a comment.

In D98940#2637166 <https://reviews.llvm.org/D98940#2637166>, @foad wrote:

>> Currently these are unbundled in SIMemoryLegalizer.

Why is the SIMemoryLegalizer the right place to do the unbundling? Why would the memory SIMemoryLegalizer know that it has "permission" to do that? The SIMemoryLegalizer was intended to do the single job of expanding the atomics semantics.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D98940



More information about the llvm-commits mailing list