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

[Robert Imschweiler via All-commits]

  Branch: refs/heads/users/ro-i/cs-chain-fp
  Home:   https://github.com/llvm/llvm-project
  Commit: de66e74ef34bb6461f0c27fb52494b08e1f3cecd
      https://github.com/llvm/llvm-project/commit/de66e74ef34bb6461f0c27fb52494b08e1f3cecd
  Author: Robert Imschweiler <robert.imschweiler at amd.com>
  Date:   2025-09-29 (Mon, 29 Sep 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

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