Improve target description check for SVE in gdbserver
authorLuis Machado <luis.machado@linaro.org>
Mon, 18 Nov 2019 19:36:53 +0000 (16:36 -0300)
committerLuis Machado <luis.machado@linaro.org>
Wed, 20 Nov 2019 16:57:34 +0000 (13:57 -0300)
commit6cdd651fda1315cb43a6a50223350da0da3f6dcf
tree75ff8739197715b1945267077bc157335556178e
parent94c9b9db4b689bc1ae643e053580db1fdfaee2d4
Improve target description check for SVE in gdbserver

The current code checks for the presence of a SVE target description by
comparing the number of registers.  This is a bit fragile since the number
of registers can change whenever we add new sets. Like PAC, for example.

If the comparison breaks, then we're left with SVE registers in the
description, but gdbserver doesn't send the registers to GDB, which in
turn displays stale information to the user.

The following patch changes the check to use the SVE feature string instead,
which hopefully should be more stable.

gdb/gdbserver/ChangeLog:

2019-11-20  Luis Machado  <luis.machado@linaro.org>

* linux-aarch64-low.c (is_sve_tdesc): Check against target feature
instead of register count.
* tdesc.c (tdesc_contains_feature): New function.
* tdesc.h (tdesc_contains_feature): New prototype.

Change-Id: I28b782cb1677560ca9a06a1be442974b25aabae4
gdb/gdbserver/ChangeLog
gdb/gdbserver/linux-aarch64-low.c
gdb/gdbserver/tdesc.c
gdb/gdbserver/tdesc.h