Clean up the XML files for ARM
Commit Message
This patch is move features/arm-*.xml to features/arm/, and it is based
on Terry's patch posted here
https://sourceware.org/ml/gdb-patches/2014-06/msg00794.html
One comment to Terry's patch is about losing "arm" prefix, and the new
patch fixes this problem.
gdb:
2016-10-04 Terry Guo <terry.guo@arm.com>
Yao Qi <yao.qi@linaro.org>
* arm-tdep.c: Adjust includes.
* features/Makefile (WHICH): Add "arm/" directory to arm
target descriptions.
(XMLTOC): Likewise.
(arm/arm-with-iwmmxt.dat): Adjust the path for
dependencies.
* features/arm-core.xml: Moved to ...
* features/arm/arm-core.xml: ... it.
* features/arm-fpa.xml: Moved to ...
* features/arm/arm-fpa.xml: ... it.
* features/arm-m-profile.xml: Moved to ...
* features/arm/arm-m-profile.xm: ... it.
* features/arm-vfpv2.xml: Moved to ...
* features/arm/arm-vfpv2.xm: ... it.
* features/arm-vfpv3.xml: Moved to ...
* features/arm/arm-vfpv3.xml: ... it.
* features/arm-with-iwmmxt.c: Moved to ...
* features/arm/arm-with-iwmmxt.c: ... it.
* features/arm-with-iwmmxt.xml: Moved to ...
* features/arm/arm-with-iwmmxt.xml: ... it.
* features/arm-with-m-fpa-layout.c: Moved to ...
* features/arm/arm-with-m-fpa-layout.c: ... it.
* features/arm-with-m-fpa-layout.xml: Moved to ...
* features/arm/arm-with-m-fpa-layout.xml: ... it.
* features/arm-with-m-vfp-d16.c: Moved to ...
* features/arm/arm-with-m-vfp-d16.c: ... it.
* features/arm-with-m-vfp-d16.xml: Moved to ...
* features/arm/arm-with-m-vfp-d16.xml: ... it.
* features/arm-with-m.c: Moved to ...
* features/arm/arm-with-m.c: ... it.
* features/arm-with-m.xml: Moved to ...
* features/arm/arm-with-m.xm: ... it.
* features/arm-with-neon.c: Moved to ...
* features/arm/arm-with-neon.c: ... it.
* features/arm-with-neon.xml: Moved to ...
* features/arm/arm-with-neon.xml: ... it.
* features/arm-with-vfpv2.c: Moved to ...
* features/arm/arm-with-vfpv2.c: ... it.
* features/arm-with-vfpv2.xml: Moved to ...
* features/arm/arm-with-vfpv2.xml: ... it.
* features/arm-with-vfpv3.c: Moved to ...
* features/arm/arm-with-vfpv3.c: ... it.
* features/arm-with-vfpv3.xml: Moved to ...
* features/arm/arm-with-vfpv3.xml: ... it.
* features/xscale-iwmmxt.xml: Moved to ...
* features/arm/xscale-iwmmxt.xml: ... it.
gdb/gdbserver:
2016-10-04 Terry Guo <terry.guo@arm.com>
Yao Qi <yao.qi@linaro.org>
* Makefile.in: Adjust the path of rules.
* configure.srv: Update the path of xml files.
* regformats/arm-with-iwmmxt.dat: Regenerated.
* regformats/arm-with-neon.dat: Likewise.
* regformats/arm-with-vfpv2.dat: Likewise.
* regformats/arm-with-vfpv3.dat Likewise.
---
gdb/arm-tdep.c | 14 +++++++-------
gdb/features/Makefile | 19 ++++++++++---------
gdb/features/{ => arm}/arm-core.xml | 0
gdb/features/{ => arm}/arm-fpa.xml | 0
gdb/features/{ => arm}/arm-m-profile.xml | 0
gdb/features/{ => arm}/arm-vfpv2.xml | 0
gdb/features/{ => arm}/arm-vfpv3.xml | 0
gdb/features/{ => arm}/arm-with-iwmmxt.c | 0
gdb/features/{ => arm}/arm-with-iwmmxt.xml | 0
gdb/features/{ => arm}/arm-with-m-fpa-layout.c | 0
gdb/features/{ => arm}/arm-with-m-fpa-layout.xml | 0
gdb/features/{ => arm}/arm-with-m-vfp-d16.c | 0
gdb/features/{ => arm}/arm-with-m-vfp-d16.xml | 0
gdb/features/{ => arm}/arm-with-m.c | 0
gdb/features/{ => arm}/arm-with-m.xml | 0
gdb/features/{ => arm}/arm-with-neon.c | 0
gdb/features/{ => arm}/arm-with-neon.xml | 0
gdb/features/{ => arm}/arm-with-vfpv2.c | 0
gdb/features/{ => arm}/arm-with-vfpv2.xml | 0
gdb/features/{ => arm}/arm-with-vfpv3.c | 0
gdb/features/{ => arm}/arm-with-vfpv3.xml | 0
gdb/features/{ => arm}/xscale-iwmmxt.xml | 0
gdb/gdbserver/Makefile.in | 16 ++++++++--------
gdb/gdbserver/configure.srv | 16 ++++++++--------
gdb/regformats/{ => arm}/arm-with-iwmmxt.dat | 2 +-
gdb/regformats/{ => arm}/arm-with-neon.dat | 2 +-
gdb/regformats/{ => arm}/arm-with-vfpv2.dat | 2 +-
gdb/regformats/{ => arm}/arm-with-vfpv3.dat | 2 +-
28 files changed, 37 insertions(+), 36 deletions(-)
rename gdb/features/{ => arm}/arm-core.xml (100%)
rename gdb/features/{ => arm}/arm-fpa.xml (100%)
rename gdb/features/{ => arm}/arm-m-profile.xml (100%)
rename gdb/features/{ => arm}/arm-vfpv2.xml (100%)
rename gdb/features/{ => arm}/arm-vfpv3.xml (100%)
rename gdb/features/{ => arm}/arm-with-iwmmxt.c (100%)
rename gdb/features/{ => arm}/arm-with-iwmmxt.xml (100%)
rename gdb/features/{ => arm}/arm-with-m-fpa-layout.c (100%)
rename gdb/features/{ => arm}/arm-with-m-fpa-layout.xml (100%)
rename gdb/features/{ => arm}/arm-with-m-vfp-d16.c (100%)
rename gdb/features/{ => arm}/arm-with-m-vfp-d16.xml (100%)
rename gdb/features/{ => arm}/arm-with-m.c (100%)
rename gdb/features/{ => arm}/arm-with-m.xml (100%)
rename gdb/features/{ => arm}/arm-with-neon.c (100%)
rename gdb/features/{ => arm}/arm-with-neon.xml (100%)
rename gdb/features/{ => arm}/arm-with-vfpv2.c (100%)
rename gdb/features/{ => arm}/arm-with-vfpv2.xml (100%)
rename gdb/features/{ => arm}/arm-with-vfpv3.c (100%)
rename gdb/features/{ => arm}/arm-with-vfpv3.xml (100%)
rename gdb/features/{ => arm}/xscale-iwmmxt.xml (100%)
rename gdb/regformats/{ => arm}/arm-with-iwmmxt.dat (91%)
rename gdb/regformats/{ => arm}/arm-with-neon.dat (92%)
rename gdb/regformats/{ => arm}/arm-with-vfpv2.dat (90%)
rename gdb/regformats/{ => arm}/arm-with-vfpv3.dat (92%)
Comments
On 10/04/2016 11:11 AM, Yao Qi wrote:
> This patch is move features/arm-*.xml to features/arm/, and it is based
> on Terry's patch posted here
> https://sourceware.org/ml/gdb-patches/2014-06/msg00794.html
>
> One comment to Terry's patch is about losing "arm" prefix, and the new
> patch fixes this problem.
This version is fine with me. Thanks.
Thanks,
Pedro Alves
On Tue, Oct 4, 2016 at 7:40 PM, Pedro Alves <palves@redhat.com> wrote:
>
> This version is fine with me. Thanks.
>
Patch is pushed in.
@@ -60,13 +60,13 @@
#include "record-full.h"
#include <algorithm>
-#include "features/arm-with-m.c"
-#include "features/arm-with-m-fpa-layout.c"
-#include "features/arm-with-m-vfp-d16.c"
-#include "features/arm-with-iwmmxt.c"
-#include "features/arm-with-vfpv2.c"
-#include "features/arm-with-vfpv3.c"
-#include "features/arm-with-neon.c"
+#include "features/arm/arm-with-m.c"
+#include "features/arm/arm-with-m-fpa-layout.c"
+#include "features/arm/arm-with-m-vfp-d16.c"
+#include "features/arm/arm-with-iwmmxt.c"
+#include "features/arm/arm-with-vfpv2.c"
+#include "features/arm/arm-with-vfpv3.c"
+#include "features/arm/arm-with-neon.c"
static int arm_debug;
@@ -44,7 +44,8 @@
# make GDB=/path/to/gdb XMLTOC="xml files" cfiles
WHICH = aarch64 \
- arm-with-iwmmxt arm-with-vfpv2 arm-with-vfpv3 arm-with-neon \
+ arm/arm-with-iwmmxt arm/arm-with-vfpv2 arm/arm-with-vfpv3 \
+ arm/arm-with-neon \
i386/i386 i386/i386-linux \
i386/i386-mmx i386/i386-mmx-linux \
i386/amd64 i386/amd64-linux \
@@ -150,13 +151,13 @@ XMLTOC = \
aarch64.xml \
arc-v2.xml \
arc-arcompact.xml \
- arm-with-iwmmxt.xml \
- arm-with-m-fpa-layout.xml \
- arm-with-m-vfp-d16.xml \
- arm-with-m.xml \
- arm-with-neon.xml \
- arm-with-vfpv2.xml \
- arm-with-vfpv3.xml \
+ arm/arm-with-iwmmxt.xml \
+ arm/arm-with-m-fpa-layout.xml \
+ arm/arm-with-m-vfp-d16.xml \
+ arm/arm-with-m.xml \
+ arm/arm-with-neon.xml \
+ arm/arm-with-vfpv2.xml \
+ arm/arm-with-vfpv3.xml \
i386/amd64-avx-linux.xml \
i386/amd64-avx.xml \
i386/amd64-avx512-linux.xml \
@@ -270,7 +271,7 @@ cfiles: $(CFILES)
sh ../../move-if-change $@.tmp $@
# Other dependencies.
-$(outdir)/arm-with-iwmmxt.dat: arm-core.xml xscale-iwmmxt.xml
+$(outdir)/arm/arm-with-iwmmxt.dat: arm/arm-core.xml arm/xscale-iwmmxt.xml
$(outdir)/i386/i386.dat: i386/32bit-core.xml i386/32bit-sse.xml
$(outdir)/i386/i386-linux.dat: i386/32bit-core.xml i386/32bit-sse.xml \
i386/32bit-linux.xml
similarity index 100%
rename from gdb/features/arm-core.xml
rename to gdb/features/arm/arm-core.xml
similarity index 100%
rename from gdb/features/arm-fpa.xml
rename to gdb/features/arm/arm-fpa.xml
similarity index 100%
rename from gdb/features/arm-m-profile.xml
rename to gdb/features/arm/arm-m-profile.xml
similarity index 100%
rename from gdb/features/arm-vfpv2.xml
rename to gdb/features/arm/arm-vfpv2.xml
similarity index 100%
rename from gdb/features/arm-vfpv3.xml
rename to gdb/features/arm/arm-vfpv3.xml
similarity index 100%
rename from gdb/features/arm-with-iwmmxt.c
rename to gdb/features/arm/arm-with-iwmmxt.c
similarity index 100%
rename from gdb/features/arm-with-iwmmxt.xml
rename to gdb/features/arm/arm-with-iwmmxt.xml
similarity index 100%
rename from gdb/features/arm-with-m-fpa-layout.c
rename to gdb/features/arm/arm-with-m-fpa-layout.c
similarity index 100%
rename from gdb/features/arm-with-m-fpa-layout.xml
rename to gdb/features/arm/arm-with-m-fpa-layout.xml
similarity index 100%
rename from gdb/features/arm-with-m-vfp-d16.c
rename to gdb/features/arm/arm-with-m-vfp-d16.c
similarity index 100%
rename from gdb/features/arm-with-m-vfp-d16.xml
rename to gdb/features/arm/arm-with-m-vfp-d16.xml
similarity index 100%
rename from gdb/features/arm-with-m.c
rename to gdb/features/arm/arm-with-m.c
similarity index 100%
rename from gdb/features/arm-with-m.xml
rename to gdb/features/arm/arm-with-m.xml
similarity index 100%
rename from gdb/features/arm-with-neon.c
rename to gdb/features/arm/arm-with-neon.c
similarity index 100%
rename from gdb/features/arm-with-neon.xml
rename to gdb/features/arm/arm-with-neon.xml
similarity index 100%
rename from gdb/features/arm-with-vfpv2.c
rename to gdb/features/arm/arm-with-vfpv2.c
similarity index 100%
rename from gdb/features/arm-with-vfpv2.xml
rename to gdb/features/arm/arm-with-vfpv2.xml
similarity index 100%
rename from gdb/features/arm-with-vfpv3.c
rename to gdb/features/arm/arm-with-vfpv3.c
similarity index 100%
rename from gdb/features/arm-with-vfpv3.xml
rename to gdb/features/arm/arm-with-vfpv3.xml
similarity index 100%
rename from gdb/features/xscale-iwmmxt.xml
rename to gdb/features/arm/xscale-iwmmxt.xml
@@ -801,14 +801,14 @@ aarch64.c : $(srcdir)/../regformats/aarch64.dat $(regdat_sh)
$(SHELL) $(regdat_sh) $(srcdir)/../regformats/aarch64.dat aarch64.c
reg-arm.c : $(srcdir)/../regformats/reg-arm.dat $(regdat_sh)
$(SHELL) $(regdat_sh) $(srcdir)/../regformats/reg-arm.dat reg-arm.c
-arm-with-iwmmxt.c : $(srcdir)/../regformats/arm-with-iwmmxt.dat $(regdat_sh)
- $(SHELL) $(regdat_sh) $(srcdir)/../regformats/arm-with-iwmmxt.dat arm-with-iwmmxt.c
-arm-with-vfpv2.c : $(srcdir)/../regformats/arm-with-vfpv2.dat $(regdat_sh)
- $(SHELL) $(regdat_sh) $(srcdir)/../regformats/arm-with-vfpv2.dat arm-with-vfpv2.c
-arm-with-vfpv3.c : $(srcdir)/../regformats/arm-with-vfpv3.dat $(regdat_sh)
- $(SHELL) $(regdat_sh) $(srcdir)/../regformats/arm-with-vfpv3.dat arm-with-vfpv3.c
-arm-with-neon.c : $(srcdir)/../regformats/arm-with-neon.dat $(regdat_sh)
- $(SHELL) $(regdat_sh) $(srcdir)/../regformats/arm-with-neon.dat arm-with-neon.c
+arm-with-iwmmxt.c : $(srcdir)/../regformats/arm/arm-with-iwmmxt.dat $(regdat_sh)
+ $(SHELL) $(regdat_sh) $(srcdir)/../regformats/arm/arm-with-iwmmxt.dat arm-with-iwmmxt.c
+arm-with-vfpv2.c : $(srcdir)/../regformats/arm/arm-with-vfpv2.dat $(regdat_sh)
+ $(SHELL) $(regdat_sh) $(srcdir)/../regformats/arm/arm-with-vfpv2.dat arm-with-vfpv2.c
+arm-with-vfpv3.c : $(srcdir)/../regformats/arm/arm-with-vfpv3.dat $(regdat_sh)
+ $(SHELL) $(regdat_sh) $(srcdir)/../regformats/arm/arm-with-vfpv3.dat arm-with-vfpv3.c
+arm-with-neon.c : $(srcdir)/../regformats/arm/arm-with-neon.dat $(regdat_sh)
+ $(SHELL) $(regdat_sh) $(srcdir)/../regformats/arm/arm-with-neon.dat arm-with-neon.c
reg-bfin.c : $(srcdir)/../regformats/reg-bfin.dat $(regdat_sh)
$(SHELL) $(regdat_sh) $(srcdir)/../regformats/reg-bfin.dat reg-bfin.c
reg-cris.c : $(srcdir)/../regformats/reg-cris.dat $(regdat_sh)
@@ -76,14 +76,14 @@ case "${target}" in
srv_tgtobj="${srv_tgtobj} arm.o"
srv_tgtobj="${srv_tgtobj} arm-linux.o"
srv_tgtobj="${srv_tgtobj} arm-get-next-pcs.o"
- srv_xmlfiles="arm-with-iwmmxt.xml"
- srv_xmlfiles="${srv_xmlfiles} arm-with-vfpv2.xml"
- srv_xmlfiles="${srv_xmlfiles} arm-with-vfpv3.xml"
- srv_xmlfiles="${srv_xmlfiles} arm-with-neon.xml"
- srv_xmlfiles="${srv_xmlfiles} arm-core.xml"
- srv_xmlfiles="${srv_xmlfiles} xscale-iwmmxt.xml"
- srv_xmlfiles="${srv_xmlfiles} arm-vfpv2.xml"
- srv_xmlfiles="${srv_xmlfiles} arm-vfpv3.xml"
+ srv_xmlfiles="arm/arm-with-iwmmxt.xml"
+ srv_xmlfiles="${srv_xmlfiles} arm/arm-with-vfpv2.xml"
+ srv_xmlfiles="${srv_xmlfiles} arm/arm-with-vfpv3.xml"
+ srv_xmlfiles="${srv_xmlfiles} arm/arm-with-neon.xml"
+ srv_xmlfiles="${srv_xmlfiles} arm/arm-core.xml"
+ srv_xmlfiles="${srv_xmlfiles} arm/xscale-iwmmxt.xml"
+ srv_xmlfiles="${srv_xmlfiles} arm/arm-vfpv2.xml"
+ srv_xmlfiles="${srv_xmlfiles} arm/arm-vfpv3.xml"
srv_linux_usrregs=yes
srv_linux_regsets=yes
srv_linux_thread_db=yes
similarity index 91%
rename from gdb/regformats/arm-with-iwmmxt.dat
rename to gdb/regformats/arm/arm-with-iwmmxt.dat
@@ -1,5 +1,5 @@
# THIS FILE IS GENERATED. -*- buffer-read-only: t -*- vi :set ro:
-# Generated from: arm-with-iwmmxt.xml
+# Generated from: arm/arm-with-iwmmxt.xml
name:arm_with_iwmmxt
xmltarget:arm-with-iwmmxt.xml
expedite:r11,sp,pc
similarity index 92%
rename from gdb/regformats/arm-with-neon.dat
rename to gdb/regformats/arm/arm-with-neon.dat
@@ -1,5 +1,5 @@
# THIS FILE IS GENERATED. -*- buffer-read-only: t -*- vi :set ro:
-# Generated from: arm-with-neon.xml
+# Generated from: arm/arm-with-neon.xml
name:arm_with_neon
xmltarget:arm-with-neon.xml
expedite:r11,sp,pc
similarity index 90%
rename from gdb/regformats/arm-with-vfpv2.dat
rename to gdb/regformats/arm/arm-with-vfpv2.dat
@@ -1,5 +1,5 @@
# THIS FILE IS GENERATED. -*- buffer-read-only: t -*- vi :set ro:
-# Generated from: arm-with-vfpv2.xml
+# Generated from: arm/arm-with-vfpv2.xml
name:arm_with_vfpv2
xmltarget:arm-with-vfpv2.xml
expedite:r11,sp,pc
similarity index 92%
rename from gdb/regformats/arm-with-vfpv3.dat
rename to gdb/regformats/arm/arm-with-vfpv3.dat
@@ -1,5 +1,5 @@
# THIS FILE IS GENERATED. -*- buffer-read-only: t -*- vi :set ro:
-# Generated from: arm-with-vfpv3.xml
+# Generated from: arm/arm-with-vfpv3.xml
name:arm_with_vfpv3
xmltarget:arm-with-vfpv3.xml
expedite:r11,sp,pc