[PATCH] Enable FeatureMP for Cortex-A5 by default

[James Molloy]

Hi Oliver,

@@ -94,7 +94,6 @@
 // RUN: %clang -target arm -mthumb -mcpu=cortex-a15 -mhwdiv=none -x c -E
-dM %s -o - | FileCheck --check-prefix=DEFAULTHWDIV-NONEHWDIV-THUMB %s
 // DEFAULTHWDIV-NONEHWDIV-THUMB-NOT:#define __ARM_ARCH_EXT_IDIV__

-
 // Check that -mfpu works properly for Cortex-A7 (enabled by default).

Looks like a whitespace diff here.

+; RUN: llc < %s -mtriple=armv7-linux-gnueabi -mcpu=cortex-a5
-mattr=-neon,+d16 | FileCheck %s --check-prefix=CORTEX-A5-NONEON

Why is there the extra +d16?

Cheers,

James


On 22 November 2013 14:19, Oliver Stannard <oliver.stannard at arm.com> wrote:

> Hi all,
>
> These patches enables FeatureMP by default for Cortex-A5 processors, and
> test that the correct build attributes are emitted when compiling for
> Cortex-A5.
>
> The reason for this is that the majority of developers using LLVM for
> Cortex-A5 will be developing "application" style software, where most
> Cortex-A5 CPUs will be Cortex-A5MP. Those developers using LLVM for
> deeply-embedded software may still need to target Cortex-A5 without the MP
> extension, but are also more likely to know the exact features of the
> hardware they are targeting, and so set the enabled features correctly. The
> same logic applies equally well to the default enabling of optional
> features
> of other Cortex-A* cores.
>
> Oliver
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20131122/70de5db4/attachment.html>