[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