This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
RE: [Patch]Clean up the XML files for ARM
- From: "Terry Guo" <terry dot guo at arm dot com>
- To: "'Will Newton'" <will dot newton at linaro dot org>
- Cc: <gdb-patches at sourceware dot org>
- Date: Wed, 11 Jun 2014 17:12:19 +0800
- Subject: RE: [Patch]Clean up the XML files for ARM
- Authentication-results: sourceware.org; auth=none
- References: <000b01cf8540$dafa6d80$90ef4880$ at arm dot com> <CANu=Dmj+bns6TtgSULCBy-5_+iLGPgY0aXOPD0pmQgSMbRuJ7g at mail dot gmail dot com>
> -----Original Message-----
> From: Will Newton [mailto:will.newton@linaro.org]
> Sent: Wednesday, June 11, 2014 4:16 PM
> To: Terry Guo
> Cc: gdb-patches@sourceware.org
> Subject: Re: [Patch]Clean up the XML files for ARM
>
> On 11 June 2014 07:46, Terry Guo <terry.guo@arm.com> wrote:
> > Hi There,
> >
> > Under folder gdb/features we are having more and more ARM related XML
> files.
> > This patch intends to move those ARM XML files into a new folder named
> arm.
> > All the references to those files are now prefixed with folder name arm.
> > Please be noted that this doesn't include AArch64 XML files so far. Is
> > it ok to GDB trunk?
> >
> > BR,
> > Terry
> >
> > 2014-06-11 Terry Guo <terry.guo@arm.com>
> >
> > * features/arm-core.xml: Move to ...
> > * features/arm/arm-core.xml: ... here.
> > * features/arm-fpa.xml: Move to ...
> > * features/arm/arm-fpa.xml: ... here.
> > * features/arm-m-profile.xml: Move to ...
> > * features/arm/arm-m-profile.xml: ... here.
> > * features/arm-vfpv2.xml: Move to ...
> > * features/arm/arm-vfpv2.xml: ... here.
> > * features/arm-vfpv3.xml: Move to ...
> > * features/arm/arm-vfpv3.xml: ... here.
> > * features/arm-with-iwmmxt.xml: Move to ...
> > * features/arm/arm-with-iwmmxt.xml: ... here.
> > * features/arm-with-iwmmxt.c: Move to ...
> > * features/arm/arm-with-iwmmxt.c: ... here.
> > * features/arm-with-m.xml: Move to ...
> > * features/arm/arm-with-m.xml: ... here.
> > * features/arm-with-m.c: Move to ...
> > * features/arm/arm-with-m.c: ... here.
> > * features/arm-with-m-fpa-layout.xml: Move to ...
> > * features/arm/arm-with-m-fpa-layout.xml: ... here.
> > * features/arm-with-m-fpa-layout.c: Move to ...
> > * features/arm/arm-with-m-fpa-layout.c: ... here.
> > * features/arm-with-m-vfp-d16.xml: Move to ...
> > * features/arm/arm-with-m-vfp-d16.xml: ... here.
> > * features/arm-with-m-vfp-d16.c: Move to ...
> > * features/arm/arm-with-m-vfp-d16.c: ... here.
> > * features/arm-with-neon.xml: Move to ...
> > * features/arm/arm-with-neon.xml: ... here.
> > * features/arm-with-neon.c: Move to ...
> > * features/arm/arm-with-neon.c: ... here.
> > * features/arm-with-vfpv2.xml: Move to ...
> > * features/arm/arm-with-vfpv2.xml: ... here.
> > * features/arm-with-vfpv2.c: Move to ...
> > * features/arm/arm-with-vfpv2.c: ... here.
> > * features/arm-with-vfpv3.xml: Move to ...
> > * features/arm/arm-with-vfpv3.xml: ... here.
> > * features/arm-with-vfpv3.c: Move to ...
> > * features/arm/arm-with-vfpv3.c: ... here.
> > * features/Makefile: Use new paths to refer ARM files.
> > * arm-tdep.c: Use new paths to include files.
>
> It seems like we have duplicate "arm" in some of the names now, it might be
> a good cleanup to rename, e.g.:
>
> features/arm/arm-vfpv3.xml -> features/arm/vfpv3.xml
>
Thanks for comments. But files in folder i386 and rs6000 are same as mine. Maybe there are reasons to do so.
For example:
i386/
i386/i386-mmx.c
i386/32bit-avx512.xml
i386/x32-avx512.xml
i386/x32-avx512-linux.c
i386/32bit-sse.xml
i386/32bit-mpx.xml
i386/i386-avx-linux.xml
i386/i386-mpx.c
i386/i386-mpx-linux.xml
i386/x32-core.xml
i386/64bit-core.xml
i386/i386.xml
./rs6000/powerpc-isa205-altivec64l.xml
./rs6000/powerpc-e500l.c
./rs6000/powerpc-altivec64l.c
./rs6000/powerpc-603.xml
./rs6000/powerpc-602.xml
./rs6000/power64-core.xml
./rs6000/powerpc-64.c
./rs6000/powerpc-altivec64.c
./rs6000/powerpc-505.xml
BR,
Terry