[all-commits] [llvm/llvm-project] c02bdd: [AMDGPU] Fix handling of FP in cs.chain functions ...

[Robert Imschweiler via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: c02bdd466a1c22221bc6de3b6817945c90979351
      https://github.com/llvm/llvm-project/commit/c02bdd466a1c22221bc6de3b6817945c90979351
  Author: Robert Imschweiler <robert.imschweiler at amd.com>
  Date:   2025-11-04 (Tue, 04 Nov 2025)

  Changed paths:
    M llvm/lib/Target/AMDGPU/SIFrameLowering.cpp
    A llvm/test/CodeGen/AMDGPU/amdgpu-cs-chain-fp-nosave.ll

  Log Message:
  -----------
  [AMDGPU] Fix handling of FP in cs.chain functions (#161194)

In case there is an dynamic alloca / an alloca which is not in the entry
block, cs.chain functions do not set up an FP, but are reported to need
one. This results in a failed assertion in
`SIFrameLowering::emitPrologue()` (Assertion `(!HasFP || FPSaved) &&
"Needed to save FP but didn't save it anywhere"' failed.) This commit
changes `hasFPImpl` so that the need for an SP in a cs.chain function
does not directly imply the need for an FP anymore.

This LLVM defect was identified via the AMD Fuzzing project.



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