[llvm] [AMDGPU] Handle expert scheduling VA_VDST WAR hazards (PR #201619)

[via llvm-commits]

================
@@ -32,8 +32,9 @@ enum InstCounterType {
   ASYNC_CNT,                         // gfx1250.
   TENSOR_CNT,                        // gfx1250.
   NUM_EXTENDED_INST_CNTS,
-  VA_VDST = NUM_EXTENDED_INST_CNTS, // gfx12+ expert mode only.
-  VM_VSRC,                          // gfx12+ expert mode only.
+  VA_VDST_RD = NUM_EXTENDED_INST_CNTS, // gfx12+ expert mode only.
+  VA_VDST_WR,                          // gfx12+ expert mode only.
----------------
vporpo wrote:

Thanks @jayfoad. I am not sure about the naming. It looks like `VA_VDST_WR` does the job of the original `VA_VDST` (that is tracking writes), but you also need a new helper/non-hardware counter to track the reads. In that case I would suggest leaving the VA_VDST counter unchanged to match the hardware counter, and name the new helper counter something completely different, like REG_READ_PSEUDO.  Just because both counters are used for deciding the wait value for S_WAITCNT_DEPCTR it doesn't mean that they need the VA_VDST prefix.

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