diff options
author | Simon Marchi <simon.marchi@efficios.com> | 2020-06-08 15:26:04 -0400 |
---|---|---|
committer | Simon Marchi <simon.marchi@efficios.com> | 2020-06-08 15:26:04 -0400 |
commit | 5d14b6e5d6525ce462c30501644922a10f8682eb (patch) | |
tree | 0f1ce729b9393dc161bbcc4fc70f5b7b0a3ff335 /gdb/mdebugread.c | |
parent | 3d967001ecd3b325fc39d7f53ebf7054d1ecd503 (diff) | |
download | gdb-5d14b6e5d6525ce462c30501644922a10f8682eb.zip gdb-5d14b6e5d6525ce462c30501644922a10f8682eb.tar.gz gdb-5d14b6e5d6525ce462c30501644922a10f8682eb.tar.bz2 |
gdb: add field::type / field::set_type
Add the `type` and `set_type` methods on `struct field`, in order to
remoremove the `FIELD_TYPE` macro. In this patch, the `FIELD_TYPE`
macro is changed to use `field::type`, so all the call sites that are
useused to set the field's type are changed to use `field::set_type`.
The next patch will remove `FIELD_TYPE` completely.
Note that because of the name clash between the existing field named
`type` and the new method, I renamed the field `m_type`. It is not
private per-se, because we can't make `struct field` a non-POD yet, but
it should be considered private anyway (not accessed outside `struct
field`).
gdb/ChangeLog:
* gdbtypes.h (struct field) <type, set_type>: New methods.
Rename `type` field to...
<m_type>: ... this. Change references throughout to use type or
set_type methods.
(FIELD_TYPE): Use field::type. Change call sites that modify
the field's type to use field::set_type instead.
Change-Id: Ie21f866e3b7f8a51ea49b722d07d272a724459a0
Diffstat (limited to 'gdb/mdebugread.c')
-rw-r--r-- | gdb/mdebugread.c | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/gdb/mdebugread.c b/gdb/mdebugread.c index 20fdd40..07613a5 100644 --- a/gdb/mdebugread.c +++ b/gdb/mdebugread.c @@ -1051,7 +1051,7 @@ parse_symbol (SYMR *sh, union aux_ext *ax, char *ext_sh, int bigend, break; SET_FIELD_ENUMVAL (*f, tsym.value); - FIELD_TYPE (*f) = t; + f->set_type (t); FIELD_NAME (*f) = debug_info->ss + cur_fdr->issBase + tsym.iss; FIELD_BITSIZE (*f) = 0; @@ -1198,7 +1198,7 @@ parse_symbol (SYMR *sh, union aux_ext *ax, char *ext_sh, int bigend, if (SYMBOL_IS_ARGUMENT (sym)) { - TYPE_FIELD_TYPE (ftype, iparams) = SYMBOL_TYPE (sym); + ftype->field (iparams).set_type (SYMBOL_TYPE (sym)); TYPE_FIELD_ARTIFICIAL (ftype, iparams) = 0; iparams++; } @@ -1238,8 +1238,8 @@ parse_symbol (SYMR *sh, union aux_ext *ax, char *ext_sh, int bigend, FIELD_NAME (*f) = name; SET_FIELD_BITPOS (*f, sh->value); bitsize = 0; - FIELD_TYPE (*f) = parse_type (cur_fd, ax, sh->index, - &bitsize, bigend, name); + f->set_type (parse_type (cur_fd, ax, sh->index, &bitsize, bigend, + name)); FIELD_BITSIZE (*f) = bitsize; } break; |