diff options
author | Luis Machado <luis.machado@linaro.org> | 2020-03-04 13:08:13 -0300 |
---|---|---|
committer | Luis Machado <luis.machado@linaro.org> | 2020-03-04 13:12:17 -0300 |
commit | 440cf44eb0f70830b8d8ac35289f84129c7a35c1 (patch) | |
tree | a9b87cebe881a6aa2abadb66b1f1a25a328a40e3 /gdb/target-descriptions.c | |
parent | 0bab6cf11655d310690e48d7c07b4aa28a87026d (diff) | |
download | gdb-440cf44eb0f70830b8d8ac35289f84129c7a35c1.zip gdb-440cf44eb0f70830b8d8ac35289f84129c7a35c1.tar.gz gdb-440cf44eb0f70830b8d8ac35289f84129c7a35c1.tar.bz2 |
Revert "gdb: Do not print empty-group regs when printing general ones"
Revert the change since it breaks existing behavior of "info registers"
for some architectures. At least AArch64 and ARM are impacted by this change.
gdb/ChangeLog:
2020-03-04 Luis Machado <luis.machado@linaro.org>
Revert aa66aac47b4dd38f9524ddb5546c08cc09930d37 due to regressions
in "info registers" for AArch64/ARM.
The change caused "info registers" to not print GPR's.
gdb/ChangeLog:
2020-02-01 Shahab Vahedi <shahab@synopsys.com>
* target-descriptions.c (tdesc_register_in_reggroup_p): Return 0
when reg->group is empty and reggroup is not.
Diffstat (limited to 'gdb/target-descriptions.c')
-rw-r--r-- | gdb/target-descriptions.c | 13 |
1 files changed, 5 insertions, 8 deletions
diff --git a/gdb/target-descriptions.c b/gdb/target-descriptions.c index 06f42a1b..04711ba 100644 --- a/gdb/target-descriptions.c +++ b/gdb/target-descriptions.c @@ -977,16 +977,13 @@ tdesc_register_in_reggroup_p (struct gdbarch *gdbarch, int regno, { struct tdesc_reg *reg = tdesc_find_register (gdbarch, regno); - if (reg != NULL) - { - if (reggroup == all_reggroup) + if (reg != NULL && !reg->group.empty () + && (reg->group == reggroup_name (reggroup))) return 1; - else if (reggroup == save_reggroup || reggroup == restore_reggroup) - return reg->save_restore; - else - return (int) (reg->group == reggroup_name (reggroup)); - } + if (reg != NULL + && (reggroup == save_reggroup || reggroup == restore_reggroup)) + return reg->save_restore; return -1; } |