[llvm-commits] [llvm] r150867 - /llvm/trunk/lib/Support/Triple.cpp

Chandler Carruth chandlerc at gmail.com
Fri Feb 17 20:34:17 PST 2012


Author: chandlerc
Date: Fri Feb 17 22:34:17 2012
New Revision: 150867

URL: http://llvm.org/viewvc/llvm-project?rev=150867&view=rev
Log:
Trivial cleanup to group the generic 'armvN' cases with the 'arm' case,
etc. No functionality changed.

Modified:
    llvm/trunk/lib/Support/Triple.cpp

Modified: llvm/trunk/lib/Support/Triple.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Support/Triple.cpp?rev=150867&r1=150866&r2=150867&view=diff
==============================================================================
--- llvm/trunk/lib/Support/Triple.cpp (original)
+++ llvm/trunk/lib/Support/Triple.cpp Fri Feb 17 22:34:17 2012
@@ -219,10 +219,6 @@
 
 Triple::ArchType Triple::ParseArch(StringRef ArchName) {
   return StringSwitch<ArchType>(ArchName)
-    // FIXME: It would be good to replace these with explicit names for all the
-    // various suffixes supported.
-    .StartsWith("armv", arm)
-    .StartsWith("thumbv", thumb)
     .Cases("i386", "i486", "i586", "i686", x86)
     .Cases("i786", "i886", "i986", x86) // FIXME: Do we need to support these?
     .Cases("amd64", "x86_64", x86_64)
@@ -230,7 +226,11 @@
     .Cases("powerpc64", "ppu", ppc64)
     .Case("mblaze", mblaze)
     .Cases("arm", "xscale", arm)
+    // FIXME: It would be good to replace these with explicit names for all the
+    // various suffixes supported.
+    .StartsWith("armv", arm)
     .Case("thumb", thumb)
+    .StartsWith("thumbv", thumb)
     .Cases("spu", "cellspu", cellspu)
     .Case("msp430", msp430)
     .Cases("mips", "mipseb", "mipsallegrex", mips)





More information about the llvm-commits mailing list