[PATCH] [AArch64] Remove NEON from "generic" CPU target

Amara Emerson amara.emerson at arm.com
Wed Nov 6 02:10:32 PST 2013


Hi t.p.northover,

Since NEON codegen support is still under development, we should not enable it for the default generic CPU target. This was exposed by r193740 which changed the default backend target to "generic" from *nothing* (which was not enabling NEON, just the default FP).

We can re-enable this when NEON is ready.



http://llvm-reviews.chandlerc.com/D2110

Files:
  lib/Target/AArch64/AArch64.td

Index: lib/Target/AArch64/AArch64.td
===================================================================
--- lib/Target/AArch64/AArch64.td
+++ lib/Target/AArch64/AArch64.td
@@ -36,8 +36,7 @@
 
 include "AArch64Schedule.td"
 
-def : Processor<"generic", GenericItineraries,
-                [FeatureFPARMv8, FeatureNEON, FeatureCrypto]>;
+def : Processor<"generic", GenericItineraries, [FeatureFPARMv8]>;
 
 //===----------------------------------------------------------------------===//
 // Register File Description
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D2110.1.patch
Type: text/x-patch
Size: 518 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20131106/4e510e16/attachment.bin>


More information about the llvm-commits mailing list