diff options
| -rw-r--r-- | gdb/ChangeLog | 6 | ||||
| -rw-r--r-- | gdb/psympriv.h | 6 | ||||
| -rw-r--r-- | gdb/symtab.h | 13 |
3 files changed, 25 insertions, 0 deletions
diff --git a/gdb/ChangeLog b/gdb/ChangeLog index 7c5efceb352..41daef6af43 100644 --- a/gdb/ChangeLog +++ b/gdb/ChangeLog @@ -1,3 +1,9 @@ +2019-11-04 Christian Biesinger <cbiesinger@google.com> + + * psympriv.h: Add static_asserts for sizeof (general_symbol_info) + and sizeof (symbol). + * symtab.h: Add a static_assert for sizeof (partial_symbol). + 2019-11-04 Rainer Orth <ro@CeBiTec.Uni-Bielefeld.DE> * NEWS (Changes since GDB 8.3): Document Solaris 10 removal. diff --git a/gdb/psympriv.h b/gdb/psympriv.h index 19d692bd230..c81261a38f6 100644 --- a/gdb/psympriv.h +++ b/gdb/psympriv.h @@ -82,6 +82,12 @@ struct partial_symbol ENUM_BITFIELD(address_class) aclass : SYMBOL_ACLASS_BITS; }; +/* This struct is size-critical (see comment at the to of symtab.h), so this + assert makes sure the size doesn't change accidentally. Be careful when + purposely increasing the size. */ +gdb_static_assert ((sizeof (void *) == 8 && sizeof (partial_symbol) == 40) + || (sizeof (void *) == 4 && sizeof (partial_symbol) == 24)); + /* A convenience enum to give names to some constants used when searching psymtabs. This is internal to psymtab and should not be used elsewhere. */ diff --git a/gdb/symtab.h b/gdb/symtab.h index eac44aee156..83b75d1647a 100644 --- a/gdb/symtab.h +++ b/gdb/symtab.h @@ -446,6 +446,13 @@ struct general_symbol_info short section; }; +/* This struct is size-critical (see comment at the top), so this assert + makes sure the size doesn't change accidentally. Be careful when + purposely increasing the size. */ +gdb_static_assert ((sizeof (void *) == 8 && sizeof (general_symbol_info) == 32) + || (sizeof (void *) == 4 && + sizeof (general_symbol_info) == 20)); + extern void symbol_set_demangled_name (struct general_symbol_info *, const char *, struct obstack *); @@ -1187,6 +1194,12 @@ struct symbol struct symbol *hash_next; }; +/* This struct is size-critical (see comment at the top), so this assert + makes sure the size doesn't change accidentally. Be careful when + purposely increasing the size. */ +gdb_static_assert ((sizeof (void *) == 8 && sizeof (symbol) == 72) + || (sizeof (void *) == 4 && sizeof (symbol) == 40)); + /* Several lookup functions return both a symbol and the block in which the symbol is found. This structure is used in these cases. */ |