================
@@ -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
_______________________________________________
llvm-branch-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
