[llvm-commits] [PATCH] Add ARM cortex-a5 subtarget

Evan Cheng evan.cheng at apple.com
Thu Nov 29 11:31:37 PST 2012


Hi Quentin,

The patch is fine. But I don't think the test case is appropriate or necessary.

Don't forget to update clang Driver/Tools.cpp and Basic/Targets.cpp.

Thanks,

Evan

On Nov 29, 2012, at 11:16 AM, Quentin Colombet <qcolombet at apple.com> wrote:

> Hello,
> 
> Here is a patch proposal to add the ARM cortex-a5 subtarget.
> It mainly consists in adding the a target description in the ARM.td file and a test case using the new cpu name, thus nothing fancy.
> 
> Cortex A5 is more or less a Cortex A8 with VFP4 extensions.
> 
> Thanks for the reviews.
> 
> Quentin
> <cortex-a5.patch>_______________________________________________
> 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