[llvm-branch-commits] [llvm] [AMDGPU] SIMemoryLegalizer: Factor out check if memory operations can affect the global AS (PR #160129)

[Fabian Ritter via llvm-branch-commits]

================
@@ -609,6 +617,15 @@ class SIGfx12CacheControl : public SIGfx11CacheControl {
   bool setAtomicScope(const MachineBasicBlock::iterator &MI,
                       SIAtomicScope Scope, SIAtomicAddrSpace AddrSpace) const;
 
+  bool canAffectGlobalAddrSpace(SIAtomicAddrSpace AS) const override {
+    assert((!ST.hasGloballyAddressableScratch() ||
----------------
ritter-x2a wrote:

You're right, yes. The split into overridden functions originated from when I thought we'd need to handle GloballyAddressableScratch in this pass, but we don't need it as long as we keep replacing the problematic scratch atomics already in AtomicExpand.
I've adjusted the PR accordingly.

https://github.com/llvm/llvm-project/pull/160129