[PATCH] D124671: [AMDGPU] Do not raise wave priority beyond a specific number of VALU instructions.

[Thomas Symalla via Phabricator via llvm-commits]

tsymalla added inline comments.


================
Comment at: llvm/lib/Target/AMDGPU/AMDGPUSetWavePriority.cpp:135
+    for (MachineInstr &MI : MBB) {
+      if (NumVALUInsts >= VALUInstsThreshold)
+        break;
----------------
You could early opt-out when `VALUInstsThreshold == 0` (at the beginning), is that correct?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D124671/new/

https://reviews.llvm.org/D124671