[llvm-branch-commits] [llvm] [AMDGPU][gfx1250] Implement SIMemoryLegalizer (PR #154726)

[Matt Arsenault via llvm-branch-commits]

================
@@ -587,7 +587,11 @@ class SIGfx12CacheControl : public SIGfx11CacheControl {
                       SIAtomicScope Scope, SIAtomicAddrSpace AddrSpace) const;
 
 public:
-  SIGfx12CacheControl(const GCNSubtarget &ST) : SIGfx11CacheControl(ST) {}
+  SIGfx12CacheControl(const GCNSubtarget &ST) : SIGfx11CacheControl(ST) {
+    // GFX12.0 and GFX12.5 memory models greatly overlap, and in some cases
+    // the behavior is the same if assuming GFX12.0 in CU mode.
+    assert(ST.hasGFX1250Insts() ? ST.isCuModeEnabled() : true);
----------------
arsenm wrote:

```suggestion
    assert(ST.hasGFX1250Insts() || ST.isCuModeEnabled());
```

https://github.com/llvm/llvm-project/pull/154726
_______________________________________________
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits