[llvm-commits] [PATCH] Add correct features for cortex-a15 in ARM.td

James Molloy james.molloy at arm.com
Mon Nov 19 02:38:07 PST 2012


Hi,

I don't think +neonfp is a correct attribute to have on as default.

+neonfp is more a "code generation strategy" than a "feature" of the
core; I think it should be user's choice whether it is used or not. It
is not 100% clear that it is the best code generation strategy in all
cases.

Cheers,

James

On Mon, 2012-11-19 at 10:02 +0000, Renato Golin wrote:
> LGTM.
> 
> On 19 November 2012 09:56, Jiangning Liu <jiangning.liu at arm.com> wrote:
> > Hi,
> >
> >
> >
> > This patch is to add correct features for ARM cortex-a15 processor.
> >
> >
> >
> > Thanks,
> >
> > -Jiangning
> >
> >
> >
> >
> > _______________________________________________
> > llvm-commits mailing list
> > llvm-commits at cs.uiuc.edu
> > http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits
> >
> 
> 
> 







More information about the llvm-commits mailing list