From: Chandler Carruth <chandlerc@gmail.com>
Date: Wed, 28 Sep 2011 09:54:07 +0000 (+0000)
Subject: Move some of the defines down to more natural locations, consolidating
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=88c75b0af17d103a116239a04dea7f27081370ab;p=clang

Move some of the defines down to more natural locations, consolidating
the target identifying macros at the top, including subtarget macros.

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

diff --git a/lib/Basic/Targets.cpp b/lib/Basic/Targets.cpp
index acf54b7334..f2404d29b1 100644
--- a/lib/Basic/Targets.cpp
+++ b/lib/Basic/Targets.cpp
@@ -1691,16 +1691,6 @@ void X86TargetInfo::getTargetDefines(const LangOptions &Opts,
     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 @@ void X86TargetInfo::getTargetDefines(const LangOptions &Opts,
     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: