[PATCH] Add support for armv6k subtarget

Renato Golin renato.golin at linaro.org
Sat Mar 7 05:43:03 PST 2015


Apart from my silly comment, looks good. I'd like Tim to have a look as well, since he has a better eye than I do for these things.


================
Comment at: include/llvm/Support/ARMBuildAttributes.h:102
@@ -101,4 +101,3 @@
   v6       = 6,   // e.g. ARM1136J_S
-  v6KZ     = 7,   // e.g. ARM1176JZ_S
-  v6T2     = 8,   // e.g. ARM1156T2F_S
-  v6K      = 9,   // e.g. ARM1136J_S
+  v6KZ     = 7,   // e.g. -
+  v6T2     = 8,   // e.g. ARM1156T2_S
----------------
That's actually ARM1176JZ_S, pure ARMv6K is more like MPCore. Ie. keep the comment. :)

http://reviews.llvm.org/D8126

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/






More information about the cfe-commits mailing list