[PATCH] Add more Cortex CPUs to Clang driver + tests
Jim Grosbach
grosbach at apple.com
Fri Sep 13 09:55:57 PDT 2013
LGTM
================
Comment at: lib/Driver/ToolChains.cpp:131
@@ -128,1 +130,3 @@
+ // See: Tools.cpp:getLLVMArchSuffixForARM()
+ .Cases("cortex-r4", "cortex-r5", "armv7")
.Case("cortex-a9-mp", "armv7f")
----------------
I would think so, yes. Not sure why it isn't. Maybe change it with a follow-up commit, assuming tests pass and such?
http://llvm-reviews.chandlerc.com/D1647
More information about the cfe-commits
mailing list