HomePhabricator

AMDGPU: Fix redundant FP spilling/assert in some functions

Authored by arsenm on Sep 21 2020, 5:43 AM.

Description

AMDGPU: Fix redundant FP spilling/assert in some functions

If a function has stack objects, and a call, we require an FP. If we
did not initially have any stack objects, and only introduced them
during PrologEpilogInserter for CSR VGPR spills, SILowerSGPRSpills
would end up spilling the FP register as if it were a normal
register. This would result in an assert in a debug build, or
redundant handling of the FP register in a release build.

Try to predict that we will have an FP later, although this is ugly.

Details

Committed
arsenmJan 26 2021, 10:01 AM
Parents
rG92d1195b5f76: AMDGPU: Add assertion to determineCalleeSaves
Branches
Unknown
Tags
Unknown