This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [ARM] Make -mcpu, -march and -mtune case-insensitive
- From: James Greenhalgh <james dot greenhalgh at arm dot com>
- To: Richard Earnshaw <Richard dot Earnshaw at arm dot com>
- Cc: Alan Lawrence <Alan dot Lawrence at arm dot com>, "gcc-patches at gcc dot gnu dot org" <gcc-patches at gcc dot gnu dot org>, Ramana Radhakrishnan <Ramana dot Radhakrishnan at arm dot com>
- Date: Fri, 17 Jan 2014 08:00:23 +0000
- Subject: Re: [ARM] Make -mcpu, -march and -mtune case-insensitive
- Authentication-results: sourceware.org; auth=none
- References: <1389888913 dot 2087 dot 19 dot camel at e104536-lin dot cambridge dot arm dot com> <52D81370 dot 1050508 at arm dot com> <20140116181333 dot GA6564 at arm dot com>
On Thu, Jan 16, 2014 at 06:13:33PM +0000, James Greenhalgh wrote:
> On Thu, Jan 16, 2014 at 05:14:24PM +0000, Richard Earnshaw wrote:
> > On 16/01/14 16:15, Alan Lawrence wrote:
> > > This is a small patch that makes the -mcpu, -march and -mtune
> > > command-line options case-insensitive, allowing e.g. -mcpu=Cortex-A15
> > > -march=ARMv7.
> > >
> > > Regression tested on arm-none-eabi with no issues; options passed onto
> > > e.g. ld are always lowercase (as previously).
> > >
> > > OK for trunk?
> > >
> > > --Alan
> > >
> > > ChangeLog:
> > > * config/arm/arm.opt: Make -mcpu, -march, -mtune case-insensitive.
> >
> > * config/arm/arm.opt (mcpu, march, mtune): Make case-insensitive.
> >
> > Don't forget the leading tab; and since someone else will have to commit
> > the patch for you, you should also include the date/author part as well.
> > Generally
> >
> > <date> Alan Lawrence <Alan.Lawrence@arm.com>
> >
> > would be acceptable, since the commit date may not be the same as the
> > posting date.
> >
> > Otherwise, this is OK.
> >
>
> I've committed this to trunk on Alan's behalf as revision 206673,
> with the following Changelog:
>
> 2014-01-16 Alan Lawrence <alan.lawrence@arm.com>
>
> * config/arm/arm.opt: Make -mcpu, -march, -mtune case-insensitive.
>
A more careful reading of your review of Alan's patch shows that this
should have been:
2014-01-16 Alan Lawrence <alan.lawrence@arm.com>
* config/arm/arm.opt (mcpu, march, mtune): Make case-insensitive.
I've fixed this up in revision 206700.
Sorry for the noise.
James