[llvm] [AMDGPU] Fix LSR cost comparator that regressed GFX9+ occupancy (PR #204344)

Yuyang Zhang via llvm-commits llvm-commits at lists.llvm.org
Tue Aug 18 23:03:19 PDT 2026


yuyzhang512 wrote:

Gentle ping. The branch is up to date with main and premerge is green.

I believe I have addressed the feedback so far:

- @rampitec — added `lsr-cost-model-occupancy.ll`. On gfx950: 260 VGPRs / occupancy 1 before, 244 VGPRs / occupancy 2 after.
- @arsenm — test shrunk to 375 lines with named values, attributes restored, and the `isNumRegsMajorCostOfLSR` comment dropped.

@arsenm @rampitec @michaelselehov — I would appreciate any guidance on what this still needs.


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


More information about the llvm-commits mailing list