This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
Re: [PATCH] Print nonexisting/optimized out static fields gracefully.
- From: Doug Evans <dje at google dot com>
- To: Pedro Alves <palves at redhat dot com>
- Cc: gdb-patches at sourceware dot org
- Date: Tue, 15 Oct 2013 16:45:33 -0700
- Subject: Re: [PATCH] Print nonexisting/optimized out static fields gracefully.
- Authentication-results: sourceware.org; auth=none
- References: <1381429510-14825-1-git-send-email-palves at redhat dot com>
Pedro Alves writes:
> With:
>
> struct static_struct { static int aaa; };
> struct static_struct sss;
> int main () { return 0; }
>
> We get:
>
> (gdb) p sss
> $1 = {static aaa = <optimized out>}
> (gdb) p sss.aaa
> field aaa is nonexistent or has been optimized out
>
> Note that the "field aaa ..." message is an error being thrown.
>
> GDB is graceful everywhere else when printing optimized out values.
> IOW it usually prints an <optimized out> value and puts that in the
> value history. I see no reason for here to be different, more so that
> when the print the whole "containing" object (well, it's a static
> field, so it's not really a container), we already print <optimized
> out>.
>
> After the patch:
>
> (gdb) p sss
> $1 = {static aaa = <optimized out>}
> (gdb) p sss.aaa
> field aaa is nonexistent or has been optimized out
> (gdb) p sss.aaa
> $2 = <optimized out>
>
> The value_entirely_optimized_out checks are there to preserve
> behavior. Without those, if the static field is a struct/union, GDB
> would go and print its fields one by one (and print <optimized out>
> for each).
>
> Tested on x86_64 Fedora 17.
>
> gdb/
> 2013-10-10 Pedro Alves <palves@redhat.com>
>
> * cp-valprint.c (cp_print_value_fields): No longer handle a NULL
> static field value.
> (cp_print_static_field): If the value is entirely optimized out,
> print <optimized out> here.
> * jv-valprint.c (java_print_value_fields): No longer handle a NULL
> static field value.
> * p-valprint.c (pascal_object_print_static_field): If the value is
> entirely optimized out, print <optimized out> here.
> * valops.c (do_search_struct_field)
> (value_struct_elt_for_reference): No longer handle a NULL static
> field value.
> * value.c (value_static_field): Return an optimized out value
> instead of NULL.
>
> gdb/testsuite/
> 2013-10-10 Pedro Alves <palves@redhat.com>
>
> * gdb.cp/m-static.exp: Adjust expected output of printing a
> nonexistent or optimized out static field. Also test printing the
> the "container" object.
"works for me"
[modulo nit noticed by Yao]