[all-commits] [llvm/llvm-project] 4c9ae7: [AMDGPU] Initialize FrameOffsetReg for amdgpu_cs_c...

[jofrn via All-commits]

  Branch: refs/heads/users/jofrn/fix-amdgpu-cs-chain-frame-pointer
  Home:   https://github.com/llvm/llvm-project
  Commit: 4c9ae7c657210e7875bfcda4f96cdd6ffd93613c
      https://github.com/llvm/llvm-project/commit/4c9ae7c657210e7875bfcda4f96cdd6ffd93613c
  Author: jofrn <jofernau at amd.com>
  Date:   2025-11-10 (Mon, 10 Nov 2025)

  Changed paths:
    M llvm/lib/Target/AMDGPU/SIMachineFunctionInfo.cpp
    A llvm/test/CodeGen/AMDGPU/amdgpu-cs-chain-frame-pointer.ll

  Log Message:
  -----------
  [AMDGPU] Initialize FrameOffsetReg for amdgpu_cs_chain functions

Functions with the amdgpu_cs_chain calling convention were not
initializing FrameOffsetReg, leaving it as FP_REG.
This caused machine code verification failures
as SCRATCH_STORE_DWORD_SADDR instructions require the saddr
operand to be in the SReg_32_XEXEC_HI register class.

This LLVM defect was identified via the AMD Fuzzing project.


  Commit: d4eecd03899caa1217f1f894eaee9cf9a15bed47
      https://github.com/llvm/llvm-project/commit/d4eecd03899caa1217f1f894eaee9cf9a15bed47
  Author: jofrn <jofernau at amd.com>
  Date:   2025-11-10 (Mon, 10 Nov 2025)

  Changed paths:
    M llvm/lib/Target/AMDGPU/SIFrameLowering.cpp
    M llvm/lib/Target/AMDGPU/SIMachineFunctionInfo.cpp
    M llvm/test/CodeGen/AMDGPU/amdgpu-cs-chain-frame-pointer.ll

  Log Message:
  -----------
  For emitEpilogue, emitCSRSpillRestores' argument in FPSaved case to handle chain functions in same way as non-FPSaved case, an offset. Remove prior hardcode to s33 and do not restore FP for chain functions.


  Commit: 820b9c3b03403a2f2f3b931648c9923cefdf8ff7
      https://github.com/llvm/llvm-project/commit/820b9c3b03403a2f2f3b931648c9923cefdf8ff7
  Author: jofrn <jofernau at amd.com>
  Date:   2025-11-10 (Mon, 10 Nov 2025)

  Changed paths:
    M llvm/lib/Target/AMDGPU/SIFrameLowering.cpp
    M llvm/test/CodeGen/AMDGPU/amdgpu-cs-chain-frame-pointer.ll

  Log Message:
  -----------
  Added test case to verify no FP setup occurs with "frame-pointer"="all".


Compare: https://github.com/llvm/llvm-project/compare/6e8105cc0553...820b9c3b0340

To unsubscribe from these emails, change your notification settings at https://github.com/llvm/llvm-project/settings/notifications