mirror of
https://github.com/darlinghq/darling-gdb.git
synced 2024-11-24 04:29:49 +00:00
[Ada] Array bound incorrectly printed for array indexed by enum subrange
Consider the following declarations: type Range_Type is (One, Two, Three); type Array_Type is array (Range_Type range One .. Two) of Integer; A : Array_Type := (1, 2); Trying to print A can yield: (gdb) print a $1 = (one => 1, 2) The bound of the first element should not have been printed, since "one" is the first enumerate of type Range_Type. Similarly, with the following declarations: type Array2_Type is array (Range_Type range Two .. Three) of Integer; A2 : Array2_Type := (2, 3); GDB is failing to print the bound of the first element of "A2": (gdb) print a2 $2 = (2, 3) This is because the index type for both types Array_Type and Array2_Type are subranges (by DWARF definition for arrays), of an anonymous subrange type. When deciding whether to print the bound of the first element, we handle subranges, but only up to one level. This patch enhanced the code to handle any number of subrange levels. gdb/ChangeLog: * ada-valprint.c (print_optional_low_bound): Get index_type's target type for as long as it is a TYPE_CODE_RANGE. No testcase with this patch, but this will be tested via the testcase of another patch, which uses the DWARF assembler to generate debugging info for an array indexed by an enum.
This commit is contained in:
parent
25790f6fc1
commit
859cf5d1de
@ -1,3 +1,8 @@
|
||||
2014-01-27 Joel Brobecker <brobecker@adacore.com>
|
||||
|
||||
* ada-valprint.c (print_optional_low_bound): Get index_type's
|
||||
target type for as long as it is a TYPE_CODE_RANGE.
|
||||
|
||||
2014-01-27 Joel Brobecker <brobecker@adacore.com>
|
||||
|
||||
* procfs.c (procfs_make_note_section): Remove assertion and
|
||||
|
@ -79,7 +79,7 @@ print_optional_low_bound (struct ui_file *stream, struct type *type,
|
||||
|
||||
index_type = TYPE_INDEX_TYPE (type);
|
||||
|
||||
if (TYPE_CODE (index_type) == TYPE_CODE_RANGE)
|
||||
while (TYPE_CODE (index_type) == TYPE_CODE_RANGE)
|
||||
{
|
||||
/* We need to know what the base type is, in order to do the
|
||||
appropriate check below. Otherwise, if this is a subrange
|
||||
|
Loading…
Reference in New Issue
Block a user