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 <[email protected]> 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 > [email protected] > http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits > >
_______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
