[cfe-commits] [llvm-commits] [PATCH] Add the Cortex-A15 target to Clang and LLVM

[Silviu Baranga]

Hi,

I've removed all the typos and formatting problems from the
LLVM patch the attached the new one.

Thanks,
Silviu

> -----Original Message-----
> From: Silviu Baranga [mailto:silbar01 at arm.com]
> Sent: 05 September 2012 16:49
> To: 'Anton Korobeynikov'
> Cc: llvm-commits at cs.uiuc.edu; cfe-commits at cs.uiuc.edu
> Subject: RE: [llvm-commits] [PATCH] Add the Cortex-A15 target to Clang
> and LLVM
> 
> Hi Anton,
> 
> The next patch wouldn't actually remove any fixme statements.
> It would remove a feature that we have determined to not be
> profitable on A15.
> 
> I will fix the typos and formatting problems and send a new
> patch (I don't know how they get through).
> 
> Thanks,
> Silviu
> 
> > -----Original Message-----
> > From: Anton Korobeynikov [mailto:anton at korobeynikov.info]
> > Sent: 05 September 2012 13:48
> > To: Silviu Baranga
> > Cc: llvm-commits at cs.uiuc.edu; cfe-commits at cs.uiuc.edu
> > Subject: Re: [llvm-commits] [PATCH] Add the Cortex-A15 target to
> Clang
> > and LLVM
> >
> > Hello Silviu,
> >
> > > keeps code generation for A15 identical to code generation for A9.
> > The
> > > patches
> > > basically enable tuning code generation for A15. Once the patches
> are
> > > accepted, we have 1 more patch in the pipeline that starts to tune
> > LLVM
> > > code generation for A15.
> > Will you please post this patch as well? It might be possible that it
> > will make sense to get rid of some FIXMEs in the present patch
> > immediately.
> >
> > You have some typos in the patch ("detemined") and also formatting
> > problems, e.g. you have stuff like
> >
> > if (foo
> >    || bar)
> >
> > Thanks!
> >
> > --
> > With best regards, Anton Korobeynikov
> > Faculty of Mathematics and Mechanics, Saint Petersburg State
> University
-------------- next part --------------
A non-text attachment was scrubbed...
Name: llvm_A15.diff
Type: application/octet-stream
Size: 15767 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20120906/ec54c609/attachment.obj>