]> git.ipfire.org Git - thirdparty/binutils-gdb.git/commit
gdb: rs6000_dwarf2_reg_to_regnum return -1 for unknown register number
authorSimon Marchi <simon.marchi@polymtl.ca>
Fri, 30 Oct 2020 19:01:13 +0000 (15:01 -0400)
committerSimon Marchi <simon.marchi@efficios.com>
Fri, 30 Oct 2020 19:01:13 +0000 (15:01 -0400)
commitaa2045e7fa41ea80a876421bca89443f7a2df92f
treed621c45ad142eb5a86769ac9798938415fe6ba09
parentb1ec273502937ca128df0ebf3e5961731fdc7272
gdb: rs6000_dwarf2_reg_to_regnum return -1 for unknown register number

With the current implementation of rs6000_dwarf2_reg_to_regnum, if an
unknown DWARF register number is passed, the same number is returned as
the internal GDB number.  This assumes that the internal GDB register
numbers match the DWARF register numbers, which is not the case.

Change it to return -1, as documented in gdbarch.sh for the
dwarf2_reg_to_regnum method.

This fixes a failure in gdb.dwarf2/bad-regnum.exp:

     (gdb) info addr foo1
    -Symbol "foo1" is a variable in $.
    -(gdb) FAIL: gdb.dwarf2/bad-regnum.exp: info addr foo1
    +Symbol "foo1" is a variable in $bad_register_number.
    +(gdb) PASS: gdb.dwarf2/bad-regnum.exp: info addr foo1

I ran the entire testsuite on powerpc64 (gcc203 on the compile farm) and
didn't see any regression.

gdb/ChangeLog:

* rs6000-tdep.c (rs6000_dwarf2_reg_to_regnum): Return -1 for
  unknown register numbers.

Change-Id: I585aa07a08f845a46c36bfdb6d3118ea94f8f54d
gdb/ChangeLog
gdb/rs6000-tdep.c