diff options
author | Yao Qi <yao.qi@linaro.org> | 2015-07-07 16:58:19 +0100 |
---|---|---|
committer | Yao Qi <yao.qi@linaro.org> | 2015-07-07 16:58:19 +0100 |
commit | 82c40d4b1deb252f2d17ff74f3fae6ae8cdd0383 (patch) | |
tree | 35a027ba1b343d5ff5c1a474e0c6763ba8cece3f /gdb/features/arm-with-m.c | |
parent | 607685ecee1015d6c37e0d800d40453dc0aadc8c (diff) | |
download | gdb-82c40d4b1deb252f2d17ff74f3fae6ae8cdd0383.zip gdb-82c40d4b1deb252f2d17ff74f3fae6ae8cdd0383.tar.gz gdb-82c40d4b1deb252f2d17ff74f3fae6ae8cdd0383.tar.bz2 |
Set architecture to arm in arm-*.xml files
This patch is to add the following line to various arm target description
xml files,
<architecture>arm</architecture>
in order to fix problems I've seen on aarch64 multi-arch debugging,
detach^M
Detaching from program: build-gdb/gdb/testsuite/gdb.base/attach, process 17145^M
(gdb) PASS: gdb.base/attach.exp: attach1 detach file^M
No executable file now.^M
Architecture of file not recognized.^M
(gdb) FAIL: gdb.base/attach.exp: attach1, purging symbols after detach
Without this patch, struct target_desc *tdesc_* are not initialised
properly, that is, fields arch and osabi in 'struct target_desc' are
not set properly. This doesn't cause any problems on single arch
debugging, because arch-utils.c:gdbarch_info_fill will guess correctly.
However, in multi-arch debugging, gdbarch_info_fill gets the aarch64
arch, but the target description is for arm (because the current
inferior is 32-bit arm).
It is a surprise to me we didn't set architecture to "arm" before in *.xml
files, and I didn't find out why didn't do so. AFAICS,
gdb/features/arm-with-iwmmxt.xml was added firstly (in patch
https://sourceware.org/ml/gdb-patches/2007-01/msg00593.html)
which had <architecture>iwmmxt</architecture>, however, afterwards,
architecture isn't set anymore in features/arm-*.xml files (in patches
https://sourceware.org/ml/gdb-patches/2009-07/msg00689.html and
https://sourceware.org/ml/gdb-patches/2010-08/msg00225.html).
gdb:
2015-07-07 Yao Qi <yao.qi@linaro.org>
* features/arm-with-m-fpa-layout.xml: Set architecture to arm.
* features/arm-with-m-fpa-layout.c: Regenerated.
* features/arm-with-m-vfp-d16.xml: Likewise.
* features/arm-with-m-vfp-d16.c: Regenerated.
* features/arm-with-m.xml: Likewise.
* features/arm-with-m.c: Regenerated.
* features/arm-with-neon.xml: Likewise.
* features/arm-with-neon.c: Regenerated.
* features/arm-with-vfpv2.xml: Likewise.
* features/arm-with-vfpv2.c: Regenerated.
* features/arm-with-vfpv3.xml: Likewise.
* features/arm-with-vfpv3.c: Regenerated.
Diffstat (limited to 'gdb/features/arm-with-m.c')
-rw-r--r-- | gdb/features/arm-with-m.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/gdb/features/arm-with-m.c b/gdb/features/arm-with-m.c index 8e27a80..64d31bb 100644 --- a/gdb/features/arm-with-m.c +++ b/gdb/features/arm-with-m.c @@ -12,6 +12,8 @@ initialize_tdesc_arm_with_m (void) struct target_desc *result = allocate_target_description (); struct tdesc_feature *feature; + set_tdesc_architecture (result, bfd_scan_arch ("arm")); + feature = tdesc_create_feature (result, "org.gnu.gdb.arm.m-profile"); tdesc_create_reg (feature, "r0", 0, 1, NULL, 32, "int"); tdesc_create_reg (feature, "r1", 1, 1, NULL, 32, "int"); |