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

[Silviu Baranga]

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