OpenE2K/binutils-gdb
12
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

Eliminate -var-create error for optzd ptr to struct

Browse Source 
This patch eliminates an error thrown when accessing the value of a
pointer to a structure where the pointer has been optimized out and
'set print object' is 'on'.  The error shows up as the rather ugly
value of the pointer variable in Eclipse.

If 'set print object' is 'on', GDB tries to determine the actual
(derived) type of the object rather than the declared type, which
requires dereferencing the pointer, which in this cases throws an
error because the pointer has been optimized out.

The fix is to simply ignore the 'print object on' setting for
pointers or references to structures when they have been optimized
out.  This means we just get the declared type instead of the actual
type, because in this case that's the best that we can do.

To implement the fix, value_optimized_out was modified so that it
no longer throws an error when it fails to fetch the specified
value.  Instead, it just checks value->optimized_out.  If we can't
definitively say that the value is optimized out, then we assume
it is not.

gdb/ChangeLog:
2016-04-06  Don Breazeal  <donb@codesourcery.com>

	* value.c (value_actual_type): Don't try to get rtti type
	of the value if it has been optimized out.
	(value_optimized_out): If a memory access error occurs,
	just check vaue->optimized_out.
This commit is contained in:
Don Breazeal 2016-04-06 14:18:31 -07:00
parent b16f8a3b1e
commit ecf2e90cd6
2 changed files with 20 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
gdb/ChangeLog
View File

@ -1,3 +1,10 @@
2016-04-06 Don Breazeal <donb@codesourcery.com>
* value.c (value_actual_type): Don't try to get rtti type
of the value if it has been optimized out.
(value_optimized_out): If a memory access error occurs,
just check vaue->optimized_out.
2016-04-06 Jan Kratochvil <jan.kratochvil@redhat.com> 2016-04-06 Jan Kratochvil <jan.kratochvil@redhat.com>
Revert the previous commit adding unknown_v_replies_ok. Revert the previous commit adding unknown_v_replies_ok.

15
gdb/value.c
View File

@ -1205,7 +1205,8 @@ value_actual_type (struct value *value, int resolve_simple_types,
if ((TYPE_CODE (result) == TYPE_CODE_PTR if ((TYPE_CODE (result) == TYPE_CODE_PTR
|| TYPE_CODE (result) == TYPE_CODE_REF) || TYPE_CODE (result) == TYPE_CODE_REF)
&& TYPE_CODE (check_typedef (TYPE_TARGET_TYPE (result))) && TYPE_CODE (check_typedef (TYPE_TARGET_TYPE (result)))
== TYPE_CODE_STRUCT) == TYPE_CODE_STRUCT
&& !value_optimized_out (value))
{ {
struct type *real_type; struct type *real_type;
@ -1433,7 +1434,17 @@ value_optimized_out (struct value *value)
/* We can only know if a value is optimized out once we have tried to /* We can only know if a value is optimized out once we have tried to
fetch it. */ fetch it. */
if (VEC_empty (range_s, value->optimized_out) && value->lazy) if (VEC_empty (range_s, value->optimized_out) && value->lazy)
value_fetch_lazy (value); {
TRY
{
value_fetch_lazy (value);
}
CATCH (ex, RETURN_MASK_ERROR)
{
/* Fall back to checking value->optimized_out. */
}
END_CATCH
}
return !VEC_empty (range_s, value->optimized_out); return !VEC_empty (range_s, value->optimized_out);
} }