]> git.ipfire.org Git - thirdparty/binutils-gdb.git/commit
include: sframe: update code comments around SFrame FRE stack offsets
authorIndu Bhagat <indu.bhagat@oracle.com>
Fri, 19 Apr 2024 23:48:17 +0000 (16:48 -0700)
committerIndu Bhagat <indu.bhagat@oracle.com>
Tue, 9 Jul 2024 17:23:31 +0000 (10:23 -0700)
commit4de9a5ccd666e218e5985d35895406f94db65f3c
tree8679be3365c28ed80b9631680ea3e996874ca816
parent6dacff8a2f4d2c24648fb96334fb272b67beff7c
include: sframe: update code comments around SFrame FRE stack offsets

This also amends the incorrect comment:
    offset3 (intrepreted as FP = CFA + offset2)

If RA tracking is enabled,  the offset to recover FP is at the third
index.  The SFrame format (V2) has assumption that if FP is saved on
stack, RA must have been saved as well.  This is true for the currently
supported arch Aarch64.  For AMD64, RA tracking per SFrame FRE is not
necessary.

In future, when extending support for more architectures, this will
likely need to be revisited.

include/
* sframe.h: Make the comments clearer by enumerating what
happens per-ABI.
include/sframe.h