[llvm-branch-commits] [llvm] [NPM] Make few more passes required (PR #203511)

[Carl Ritson via llvm-branch-commits]

================
@@ -14,7 +14,7 @@
 namespace llvm {
 
 class AMDGPUWaitSGPRHazardsPass
-    : public OptionalPassInfoMixin<AMDGPUWaitSGPRHazardsPass> {
+    : public RequiredPassInfoMixin<AMDGPUWaitSGPRHazardsPass> {
----------------
perlfu wrote:

This pass is required, but only on GFX11 and GFX12.  It replaces the hazard recognizer for this hazard class because the architecture of the standard hazard recognizer is in appropriate for addressing these.

Option to disable it exists for performance evaluation and/or tuning small kernels.
This was explained during review: https://github.com/llvm/llvm-project/pull/118750#discussion_r1912799620

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