[llvm] [AMDGPU] Fix GFX1250 hazard: S_SET_VGPR_MSB dropped (PR #184904)

Stanislav Mekhanoshin via llvm-commits llvm-commits at lists.llvm.org
Fri Mar 6 02:44:28 PST 2026


rampitec wrote:

> > It's a huge change and hazard recognizer mitigates the hazard for instruction which does not exist yet. Why not to insert nops in the VGPR lowering pass itself?
> 
> Why not swap the order of passes so that GCNHazardRcognizer runs after AMDGPULowerVGPREncoding? That's how I assumed it was already. That's why I suggested fixing it in the hazard recognizer.

Because contract with HW was that this MSB lowering does not create hazards in the first place.

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


More information about the llvm-commits mailing list