libgcc: Move cfa_how into potential padding in struct frame_state_reg_info

Message ID 874jw6mk1s.fsf@oldenburg.str.redhat.com
State Accepted
Headers
Series libgcc: Move cfa_how into potential padding in struct frame_state_reg_info |

Checks

Context Check Description
snail/gcc-patch-check success Github commit url

Commit Message

Florian Weimer Oct. 14, 2022, 4:44 p.m. UTC
  On many architectures, there is a padding gap after the how array
member, and cfa_how can be moved there.  This reduces the size of the
struct and the amount of memory that uw_frame_state_for has to clear.

There is no measurable performance benefit from this on x86-64 (even
though the memset goes from 120 to 112 bytes), but it seems to be a
good idea to do anyway.

libgcc/

	* unwind-dw2.h (struct frame_state_reg_info): Move cfa_how member
	and reduce its size.

---
 libgcc/unwind-dw2.h | 11 ++++++-----
 1 file changed, 6 insertions(+), 5 deletions(-)


base-commit: 3dfeda095bd43c011fdc3834b9cec39bb9a73a1f
  

Comments

Richard Sandiford Oct. 14, 2022, 6 p.m. UTC | #1
Florian Weimer via Gcc-patches <gcc-patches@gcc.gnu.org> writes:
> On many architectures, there is a padding gap after the how array
> member, and cfa_how can be moved there.  This reduces the size of the
> struct and the amount of memory that uw_frame_state_for has to clear.
>
> There is no measurable performance benefit from this on x86-64 (even
> though the memset goes from 120 to 112 bytes), but it seems to be a
> good idea to do anyway.
>
> libgcc/
>
> 	* unwind-dw2.h (struct frame_state_reg_info): Move cfa_how member
> 	and reduce its size.

OK, thanks.

Richard

>
> ---
>  libgcc/unwind-dw2.h | 11 ++++++-----
>  1 file changed, 6 insertions(+), 5 deletions(-)
>
> diff --git a/libgcc/unwind-dw2.h b/libgcc/unwind-dw2.h
> index af34e000f07..a0834b18277 100644
> --- a/libgcc/unwind-dw2.h
> +++ b/libgcc/unwind-dw2.h
> @@ -50,6 +50,12 @@ typedef struct
>      } reg[__LIBGCC_DWARF_FRAME_REGISTERS__+1];
>      unsigned char how[__LIBGCC_DWARF_FRAME_REGISTERS__+1];
>  
> +    enum {
> +      CFA_UNSET,
> +      CFA_REG_OFFSET,
> +      CFA_EXP
> +    } cfa_how : 8;
> +
>      /* Used to implement DW_CFA_remember_state.  */
>      struct frame_state_reg_info *prev;
>  
> @@ -58,11 +64,6 @@ typedef struct
>      _Unwind_Sword cfa_offset;
>      _Unwind_Word cfa_reg;
>      const unsigned char *cfa_exp;
> -    enum {
> -      CFA_UNSET,
> -      CFA_REG_OFFSET,
> -      CFA_EXP
> -    } cfa_how;
>    } regs;
>  
>    /* The PC described by the current frame state.  */
>
> base-commit: 3dfeda095bd43c011fdc3834b9cec39bb9a73a1f
  

Patch

diff --git a/libgcc/unwind-dw2.h b/libgcc/unwind-dw2.h
index af34e000f07..a0834b18277 100644
--- a/libgcc/unwind-dw2.h
+++ b/libgcc/unwind-dw2.h
@@ -50,6 +50,12 @@  typedef struct
     } reg[__LIBGCC_DWARF_FRAME_REGISTERS__+1];
     unsigned char how[__LIBGCC_DWARF_FRAME_REGISTERS__+1];
 
+    enum {
+      CFA_UNSET,
+      CFA_REG_OFFSET,
+      CFA_EXP
+    } cfa_how : 8;
+
     /* Used to implement DW_CFA_remember_state.  */
     struct frame_state_reg_info *prev;
 
@@ -58,11 +64,6 @@  typedef struct
     _Unwind_Sword cfa_offset;
     _Unwind_Word cfa_reg;
     const unsigned char *cfa_exp;
-    enum {
-      CFA_UNSET,
-      CFA_REG_OFFSET,
-      CFA_EXP
-    } cfa_how;
   } regs;
 
   /* The PC described by the current frame state.  */