[llvm] [AMDGPU] Handle expert scheduling VA_VDST WAR hazards (PR #201619)
Jay Foad via llvm-commits
llvm-commits at lists.llvm.org
Fri Jul 17 01:13:00 PDT 2026
================
@@ -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.
----------------
jayfoad wrote:
> On the other hand VA_VDST_RD is a "made up" counter that has no hardware equivalent,
No, I completely disagree with this description. There is one hardware counter. The _only_ reason for having two separate `InstCounterType`s is that we want to remember two different values of this counter for each VGPR. But neither of these two uses is more "real" or "made up" than the other.
> I don't want to block this, feel free to merge. We can revisit the names if needed in a future patch.
Thanks :)
https://github.com/llvm/llvm-project/pull/201619
More information about the llvm-commits
mailing list