[llvm] [AMDGPU] Refactor hazard recognizer for VALU-pipeline hazards. NFCI. (PR #168801)

[Stanislav Mekhanoshin via llvm-commits]

================
@@ -94,6 +98,9 @@ class GCNHazardRecognizer final : public ScheduleHazardRecognizer {
   int checkReadM0Hazards(MachineInstr *SMovRel);
   int checkNSAtoVMEMHazard(MachineInstr *MI);
   int checkFPAtomicToDenormModeHazard(MachineInstr *MI);
+  // emit V_NOP instructions. \p WaitStatesNeeded is the number of V_NOPs we
+  // need to insert, negative means not needed.
----------------
rampitec wrote:

I agree, but all other `check*` functions return `int` as well.

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