This is the mail archive of the gdb-patches@sourceware.org mailing list for the GDB project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [PATCH] dwarf2read.c (read_structure_type): Delete outdated comments


On Wed, Apr 16, 2014 at 12:52 PM, Doug Evans <dje@google.com> wrote:
> Hi.
>
> This patch could use another set of eyes.
>
> The comment here is (ISTM) just plain wrong (or at best misleading).
> We don't call new_symbol here, that is done by process_structure_scope.
>
>
> 2014-04-16  Doug Evans  <dje@google.com>
>
>         * dwarf2read.c (read_structure_type): Delete outdated comments.
>
> diff --git a/gdb/dwarf2read.c b/gdb/dwarf2read.c
> index 6d1b90c..c0ca557 100644
> --- a/gdb/dwarf2read.c
> +++ b/gdb/dwarf2read.c
> @@ -12800,19 +12800,14 @@ producer_is_icc (struct dwarf2_cu *cu)
>  /* Called when we find the DIE that starts a structure or union scope
>     (definition) to create a type for the structure or union.  Fill in
>     the type's name and general properties; the members will not be
> -   processed until process_structure_scope.
> +   processed until process_structure_scope.  A symbol table entry for
> +   the type will also not be done until process_structure_scope (assuming
> +   the type has a name).
>
>     NOTE: we need to call these functions regardless of whether or not the
>     DIE has a DW_AT_name attribute, since it might be an anonymous
>     structure or union.  This gets the type entered into our set of
> -   user defined types.
> -
> -   However, if the structure is incomplete (an opaque struct/union)
> -   then suppress creating a symbol table entry for it since gdb only
> -   wants to find the one with the complete definition.  Note that if
> -   it is complete, we just call new_symbol, which does it's own
> -   checking about whether the struct/union is anonymous or not (and
> -   suppresses creating a symbol table entry itself).  */
> +   user defined types.  */
>
>  static struct type *
>  read_structure_type (struct die_info *die, struct dwarf2_cu *cu)

fyi,
I have committed this.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]