diff --git a/gdb/ChangeLog b/gdb/ChangeLog index ca9f364745..6ea59592fa 100644 --- a/gdb/ChangeLog +++ b/gdb/ChangeLog @@ -1,3 +1,9 @@ +2020-02-18 Simon Marchi + + * dwarf2/read.c (update_enumeration_type_from_children): Allow + flag enums to contain duplicate enumerators. + * valprint.c (generic_val_print_enum_1): Update comment. + 2020-02-18 Simon Marchi * dwarf2/read.c: Include "count-one-bits.h". diff --git a/gdb/dwarf2/read.c b/gdb/dwarf2/read.c index 5a77b62b5a..ee220ab70b 100644 --- a/gdb/dwarf2/read.c +++ b/gdb/dwarf2/read.c @@ -15495,7 +15495,6 @@ update_enumeration_type_from_children (struct die_info *die, struct die_info *child_die; int unsigned_enum = 1; int flag_enum = 1; - ULONGEST mask = 0; auto_obstack obstack; @@ -15531,10 +15530,6 @@ update_enumeration_type_from_children (struct die_info *die, { if (count_one_bits_ll (value) >= 2) flag_enum = 0; - else if ((mask & value) != 0) - flag_enum = 0; - else - mask |= value; } /* If we already know that the enum type is neither unsigned, nor diff --git a/gdb/testsuite/ChangeLog b/gdb/testsuite/ChangeLog index 5aaf5feecd..4f63dde12a 100644 --- a/gdb/testsuite/ChangeLog +++ b/gdb/testsuite/ChangeLog @@ -1,3 +1,8 @@ +2020-02-18 Simon Marchi + + * gdb.base/printcmds.c (enum flag_enum): Add FE_TWO_LEGACY + enumerator. + 2020-02-18 Simon Marchi * gdb.base/printcmds.c (enum flag_enum): Prefix enumerators with diff --git a/gdb/testsuite/gdb.base/printcmds.c b/gdb/testsuite/gdb.base/printcmds.c index acb3cb3ad2..ed1e26b12a 100644 --- a/gdb/testsuite/gdb.base/printcmds.c +++ b/gdb/testsuite/gdb.base/printcmds.c @@ -99,9 +99,10 @@ volatile enum some_volatile_enum some_volatile_enum = enumvolval1; /* An enum considered as a "flag enum". */ enum flag_enum { - FE_NONE = 0x00, - FE_ONE = 0x01, - FE_TWO = 0x02, + FE_NONE = 0x00, + FE_ONE = 0x01, + FE_TWO = 0x02, + FE_TWO_LEGACY = 0x02, }; enum flag_enum three = FE_ONE | FE_TWO; diff --git a/gdb/valprint.c b/gdb/valprint.c index 77b9a4993d..888c9cdb57 100644 --- a/gdb/valprint.c +++ b/gdb/valprint.c @@ -631,9 +631,10 @@ generic_val_print_enum_1 (struct type *type, LONGEST val, { int first = 1; - /* We have a "flag" enum, so we try to decompose it into - pieces as appropriate. A flag enum has disjoint - constants by definition. */ + /* We have a "flag" enum, so we try to decompose it into pieces as + appropriate. The enum may have multiple enumerators representing + the same bit, in which case we choose to only print the first one + we find. */ fputs_filtered ("(", stream); for (i = 0; i < len; ++i) {