This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

RE: [PING] iwMMXt patches


> From: Matt Turner [mailto:mattst88@gmail.com]
> To: Xinyu Qi
> Cc: Ramana Radhakrishnan; GCC Patches
> Subject: Re: [PING] iwMMXt patches
> 
> On Tue, Apr 17, 2012 at 4:17 PM, Matt Turner <mattst88@gmail.com> wrote:
> > Are these patches ready to go in? It looks like they were ack'd.
> >
> > http://gcc.gnu.org/ml/gcc-patches/2011-10/msg01815.html
> > http://gcc.gnu.org/ml/gcc-patches/2011-10/msg01817.html
> > http://gcc.gnu.org/ml/gcc-patches/2011-10/msg01816.html
> > http://gcc.gnu.org/ml/gcc-patches/2011-10/msg01818.html
> > http://gcc.gnu.org/ml/gcc-patches/2011-10/msg01819.html
> >
> > We (OLPC) will need these patches for reasonable iwMMXt performance
> > and the ability to use VFP and iwMMXt together.
> >
> > Thanks,
> > Matt
> 
> Xinyu,
> 
> With these patches I don't see a new -mcpu flag. Isn't a tune/cpu flag
> the normal way to activate this code?
> 
> Other .md files have statements like (eq_attr "tune" "cortexa8"), but
> I don't see how to turn on the marvell-f-iwmmxt attribute, ie (eq_attr
> "marvell_f_iwmmxt" "yes").
> 
> Please let me know.
> 
> Thanks,
> Matt

Hi Matt,

I updated the patches several months ago by following the review opinions form Richard Earnshaw [Richard.Earnshaw@arm.com]
(unfortunately, no further feedback)
The newest patches are 
http://gcc.gnu.org/ml/gcc-patches/2011-12/msg01787.html
http://gcc.gnu.org/ml/gcc-patches/2011-12/msg01788.html
http://gcc.gnu.org/ml/gcc-patches/2011-12/msg01789.html
http://gcc.gnu.org/ml/gcc-patches/2011-12/msg01786.html
http://gcc.gnu.org/ml/gcc-patches/2011-12/msg01599.html
The main discussion is in
http://gcc.gnu.org/ml/gcc-patches/2011-12/msg01786.html

No new -mcpu flag is introduced in the patches. You can simply turn on marvell-f-iwmmxt by -mcpu=iwmmxt2(or -march=iwmmxt2).
(Of course it is odd to treat the "iwmmxt2" as a name of cpu)


Thanks,
Xinyu


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]