[PATCH] Move logic from llvm::Triple::getARMCPUForArch to driver::tools::arm::getARMCPUForMArch

Derek Schuff dschuff at google.com
Thu Mar 26 09:44:05 PDT 2015


The out of line function is on the LLVM side, so that will be a separate commit (http://reviews.llvm.org/D8637)


================
Comment at: lib/Driver/Tools.cpp:5544
@@ -5543,2 +5543,3 @@
 
-  return Triple.getARMCPUForArch(MArch);
+  switch (Triple.getOS()) {
+  case llvm::Triple::FreeBSD:
----------------
rengolin wrote:
> Move this switch inside the identical one below, after the StringSwitch.
I don't think that will work. This logic is supposed to override the StringSwitch; e.g. if the OS is FreeBSD and MArch is "armv6" then the CPU is supposed to be "arm1176jzf-s" but if this switch is moved, then the StringSwitch logic will return "arm1136jf-s"

http://reviews.llvm.org/D8622

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






More information about the cfe-commits mailing list