[PATCH] [ARM] Add support for Cortex-A17

James Molloy james.molloy at arm.com
Fri Oct 10 05:52:18 PDT 2014


Hi Matt,

This looks good to me, provided you fix the indentation I mention in an inline comment.

Cheers,

James

================
Comment at: lib/Target/ARM/ARM.td:380
@@ +379,3 @@
+def : ProcessorModel<"cortex-a17", CortexA9Model,
+                                    [ProcA17, HasV7Ops, FeatureNEON, FeatureDB,
+                                     FeatureDSPThumb2, FeatureMP,
----------------
This should be aligned to the first '"' character in "cortex-a17".

http://reviews.llvm.org/D5725






More information about the llvm-commits mailing list