[llvm] [AMDGPU] Initialize FrameOffsetReg for amdgpu_cs_chain functions (PR #165518)

via llvm-commits llvm-commits at lists.llvm.org
Mon Nov 17 07:29:51 PST 2025


jofrn wrote:

> This should definitely work on GFX12 and -O0! If you feel like the issues are not related to your patch, let me know and I can look into fixing them. Thanks :)

It can fail with or without -O0; I had a typo, clarified above.

The current issue occurs from the fp_all test case. The original patch for recurse requires only

`return !MF.getInfo<SIMachineFunctionInfo>()->isChainFunction() &&
         (frameTriviallyRequiresSP(MFI) || MFI.isFrameAddressTaken() ...`

fp_all requires perhaps the additional modifications to `SIFrameLowering::emitEpilogue`.

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


More information about the llvm-commits mailing list