[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