[PATCH] D99061: [AMDGPU] Only unbundle memory accesses in SIMemoryLegalizer

[Carl Ritson via Phabricator via llvm-commits]

critson added a comment.

In D99061#2641967 <https://reviews.llvm.org/D99061#2641967>, @t-tye wrote:

> What was the decision on the philosophy on who should be doing unbundling in general? How is it known that it is sfe for the memory legalized to do this unbundling? Is the policy documented in some AMD GPU design page?

I do not think we have reached a consensus yet.
However my understanding of current direction is:

- Stop all unbundling where possible
- Add annotations to bundles so that downstream passes know which ones are legal to modify


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D99061