[PATCH] D120279: [AMDGPU] Extend SILoadStoreOptimizer to handle global loads

[Matt Arsenault via Phabricator via llvm-commits]

arsenm added a comment.

Overall I think this pass should do less than it currently does. All of these cases should have been handled in the IR already, it's not the disjoint case of the DS operations.

This is another example where I think having constant address space is overcomplicated and would be better served by global loads marked invariant


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

https://reviews.llvm.org/D120279