MIPS: Added support for Loongson architectures.

src/mips/constants-mips.h

Index: src/mips/constants-mips.h
diff --git a/src/mips/constants-mips.h b/src/mips/constants-mips.h
index d62a8901f0ad0a0ffe497a9b8e655759fc4a9b7a..4e710d2bad7e8f66a407b7c5101418fddf81a3d6 100644
--- a/src/mips/constants-mips.h
+++ b/src/mips/constants-mips.h
@@ -39,11 +39,16 @@
 
 #define UNSUPPORTED_MIPS() v8::internal::PrintF("Unsupported instruction.\n")
 
+enum arch_variants {mips32r2, mips32r1, loongson};

[Yang, 2012/02/22 10:58:15]

I strongly suggest using uppercase identifiers for enumerations, as we do
elsewhere in V8 as well.

[Kevin Millikin (Google), 2012/02/22 11:29:39]

Good catch.  The style recommendation is to capitalize the type name
(ArchVariants) and use either kLoongson or LOONGSON style for the value names.

http://google-styleguide.googlecode.com/svn/trunk/cppguide.xml?showone=Type_N...

http://google-styleguide.googlecode.com/svn/trunk/cppguide.xml?showone=Enumer...

[kalmard, 2012/02/23 13:23:47]

Done.

 
 #ifdef _MIPS_ARCH_MIPS32R2
-  #define mips32r2 1
+  static const arch_variants arch_variant = mips32r2;
+#elif _MIPS_ARCH_LOONGSON
+// The loongson flag refers to the LOONGSON architectures based on MIPS-III,
+// which predates (and is a subset of) the mips32r2 and r1 architectures.
+  static const arch_variants arch_variant = loongson;
 #else
-  #define mips32r2 0
+  static const arch_variants arch_variant = mips32r1;
 #endif