[mips] Treat -mcpu=generic the same way as an empty CPU string.
authorDaniel Sanders <daniel.sanders@imgtec.com>
Wed, 26 Feb 2014 10:20:15 +0000 (10:20 +0000)
committerDaniel Sanders <daniel.sanders@imgtec.com>
Wed, 26 Feb 2014 10:20:15 +0000 (10:20 +0000)
Summary:
This should fix the MCJIT unit tests that were broken by r201792 on the MIPS buildbot.
MIPS currently uses the default implementation of sys::getHostCPUName() which
always returns "generic". For now, we will accept "generic" and coerce it to
"mips32" or "mips64" depending on the target architecture like we do for empty
CPU names.

Reviewers: jacksprat, matheusalmeida

Reviewed By: jacksprat

Differential Revision: http://llvm-reviews.chandlerc.com/D2878

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@202253 91177308-0d34-0410-b5e6-96231b3b80d8

lib/Target/Mips/MCTargetDesc/MipsMCTargetDesc.cpp
lib/Target/Mips/MipsSubtarget.cpp

index 637b668f67cd7c8397259603d46c2f429e88009d..faecde41b069d704ae253a4128c15e9a15268514 100644 (file)
@@ -42,7 +42,7 @@ using namespace llvm;
 /// Select the Mips CPU for the given triple and cpu name.
 /// FIXME: Merge with the copy in MipsSubtarget.cpp
 static inline StringRef selectMipsCPU(StringRef TT, StringRef CPU) {
-  if (CPU.empty()) {
+  if (CPU.empty() || CPU == "generic") {
     Triple TheTriple(TT);
     if (TheTriple.getArch() == Triple::mips ||
         TheTriple.getArch() == Triple::mipsel)
index f16fb798286782699422630e28b3794cc6aa1e29..af83d61f091941261dcfa6939d54726902fabca6 100644 (file)
@@ -62,7 +62,7 @@ Mips16ConstantIslands(
 /// Select the Mips CPU for the given triple and cpu name.
 /// FIXME: Merge with the copy in MipsMCTargetDesc.cpp
 static inline StringRef selectMipsCPU(StringRef TT, StringRef CPU) {
-  if (CPU.empty()) {
+  if (CPU.empty() || CPU == "generic") {
     Triple TheTriple(TT);
     if (TheTriple.getArch() == Triple::mips ||
         TheTriple.getArch() == Triple::mipsel)