[cfe-commits] r140689 - /cfe/trunk/lib/Basic/Targets.cpp

[Chandler Carruth]

Author: chandlerc
Date: Wed Sep 28 04:54:07 2011
New Revision: 140689

URL: http://llvm.org/viewvc/llvm-project?rev=140689&view=rev
Log:
Move some of the defines down to more natural locations, consolidating
the target identifying macros at the top, including subtarget macros.

Modified:
    cfe/trunk/lib/Basic/Targets.cpp

Modified: cfe/trunk/lib/Basic/Targets.cpp
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Basic/Targets.cpp?rev=140689&r1=140688&r2=140689&view=diff
==============================================================================
--- cfe/trunk/lib/Basic/Targets.cpp (original)
+++ cfe/trunk/lib/Basic/Targets.cpp Wed Sep 28 04:54:07 2011
@@ -1691,16 +1691,6 @@
     DefineStd(Builder, "i386", Opts);
   }
 
-  if (HasAES)
-    Builder.defineMacro("__AES__");
-
-  if (HasAVX)
-    Builder.defineMacro("__AVX__");
-
-  // Target properties.
-  Builder.defineMacro("__LITTLE_ENDIAN__");
-  Builder.defineMacro("__REGISTER_PREFIX__", "");
-
   // Subtarget options.
   // FIXME: We are hard-coding the tune parameters based on the CPU, but they
   // truly should be based on -mtune options.
@@ -1831,11 +1821,21 @@
     break;
   }
 
+  // Target properties.
+  Builder.defineMacro("__LITTLE_ENDIAN__");
+  Builder.defineMacro("__REGISTER_PREFIX__", "");
+
   // Define __NO_MATH_INLINES on linux/x86 so that we don't get inline
   // functions in glibc header files that use FP Stack inline asm which the
   // backend can't deal with (PR879).
   Builder.defineMacro("__NO_MATH_INLINES");
 
+  if (HasAES)
+    Builder.defineMacro("__AES__");
+
+  if (HasAVX)
+    Builder.defineMacro("__AVX__");
+
   // Each case falls through to the previous one here.
   switch (SSELevel) {
   case SSE42: