diff options
author | Pedro Franco de Carvalho <pedromfc@linux.ibm.com> | 2019-01-14 17:28:53 -0200 |
---|---|---|
committer | Pedro Franco de Carvalho <pedromfc@linux.ibm.com> | 2019-01-14 17:28:53 -0200 |
commit | 6f072a103498fa9b7795782d9fdf1ae226f5565e (patch) | |
tree | db24ed766ad9899ab082fb730ee5599a5e13daa3 /gdb/ppc-tdep.h | |
parent | a7b8d68257c52281b08d9d858805e15667be6606 (diff) | |
download | binutils-6f072a103498fa9b7795782d9fdf1ae226f5565e.zip binutils-6f072a103498fa9b7795782d9fdf1ae226f5565e.tar.gz binutils-6f072a103498fa9b7795782d9fdf1ae226f5565e.tar.bz2 |
[PowerPC] Aliases for vector registers
This patch defines pseudo-registers "v0" through "v31" as aliases that
map to the corresponding raw "vr0" through "vr31" vector registers for
Power.
The motivation behind this is that although GDB defines these
registers as "vrX", the disassembler prints them as "vX", e.g. as the
operands in instructions such as "vaddubm v2,v1,v1". This can be
confusing to users trying to print out the values of the operands
while inspecting the disassembled code.
The new aliases are made not to belong to any register group, to avoid
duplicated values in "info register vector" and "info register all".
The arch-specific rs6000_pseudo_register_reggroup_p function had
previously been removed since the other pseudo-registers could have
their groups inferred by their type. It restored with this patch to
handle the aliases. Membership for the other pseudo-registers is
still determined using the default function.
A new tests checks that GDB prints the expected values of vector
registers after they are filled by the inferior, by using both the raw
names and the aliases. Two other existing tests are modified to also
test the aliases.
gdb/ChangeLog:
2019-01-14 Pedro Franco de Carvalho <pedromfc@linux.ibm.com>
* ppc-tdep.h (struct gdbarch_tdep) <ppc_v0_alias_regnum>: New
field.
* rs6000-tdep.c: Include reggroups.h.
(IS_V_ALIAS_PSEUDOREG): Define.
(rs6000_register_name): Return names for the "vX" aliases.
(rs6000_pseudo_register_type): Return type for the "vX" aliases.
(rs6000_pseudo_register_reggroup_p): Restore. Handle "vX"
aliases. Call default_register_reggroup_p for all other
pseudo-registers.
(v_alias_pseudo_register_read, v_alias_pseudo_register_write):
New functions.
(rs6000_pseudo_register_read, rs6000_pseudo_register_write):
Handle "vX" aliases.
(v_alias_pseudo_register_collect): New function.
(rs6000_ax_pseudo_register_collect): Handle "vX" aliases.
(rs6000_gdbarch_init): Initialize "vX" aliases as
pseudo-registers. Restore registration of
rs6000_pseudo_register_reggroup_p with
set_tdesc_pseudo_register_reggroup_p.
gdb/testsuite/ChangeLog:
2019-01-14 Pedro Franco de Carvalho <pedromfc@linux.ibm.com>
* gdb.arch/vsx-regs.exp: Add tests that use the vector register
aliases.
* gdb.arch/altivec-regs.exp: Likewise. Fix indentation of two
tests.
* gdb.arch/powerpc-vector-regs.c: New file.
* gdb.arch/powerpc-vector-regs.exp: New file.
gdb/doc/ChangeLog:
2019-01-14 Pedro Franco de Carvalho <pedromfc@linux.ibm.com>
* gdb.texinfo (PowerPC Features): Document the alias
pseudo-registers for the org.gnu.gdb.power.altivec feature.
Diffstat (limited to 'gdb/ppc-tdep.h')
-rw-r--r-- | gdb/ppc-tdep.h | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/gdb/ppc-tdep.h b/gdb/ppc-tdep.h index ff3c35ee..9297fe2 100644 --- a/gdb/ppc-tdep.h +++ b/gdb/ppc-tdep.h @@ -244,6 +244,10 @@ struct gdbarch_tdep int ppc_vr0_regnum; /* First AltiVec register. */ int ppc_vrsave_regnum; /* Last AltiVec register. */ + /* Altivec pseudo-register vX aliases for the raw vrX + registers. */ + int ppc_v0_alias_regnum; + /* SPE registers. */ int ppc_ev0_upper_regnum; /* First GPR upper half register. */ int ppc_ev0_regnum; /* First ev register. */ |