Only force SSE level if it is not correct.
authorTorok Edwin <edwintorok@gmail.com>
Mon, 2 Feb 2009 21:57:34 +0000 (21:57 +0000)
committerTorok Edwin <edwintorok@gmail.com>
Mon, 2 Feb 2009 21:57:34 +0000 (21:57 +0000)
Add an assert to check HasX86_64 status.

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

lib/Target/X86/X86Subtarget.cpp

index 99708e04cf9c5f61835fe43a6d35a36613d0d1ea..185f45b4cf1b5bcf5bbc79065c3c7751d7111300 100644 (file)
@@ -322,16 +322,23 @@ X86Subtarget::X86Subtarget(const Module &M, const std::string &FS, bool is64Bit)
     // If feature string is not empty, parse features string.
     std::string CPU = GetCurrentX86CPU();
     ParseSubtargetFeatures(FS, CPU);
+    // All X86-64 CPUs also have SSE2, however user might request no SSE via 
+    // -mattr, so don't force SSELevel here.
   } else {
     // Otherwise, use CPUID to auto-detect feature set.
     AutoDetectSubtargetFeatures();
+    if (Is64Bit && X86SSELevel < SSE2) {
+      // Make sure SSE2  is enabled, it is available on all X86-64 CPUs.
+      X86SSELevel = SSE2;
+    }
   }
     
-  // If requesting codegen for X86-64, make sure that 64-bit and SSE2 features
-  // are enabled.  These are available on all x86-64 CPUs.
+  // If requesting codegen for X86-64, make sure that 64-bit features
+  // are enabled.  
   if (Is64Bit) {
-    HasX86_64 = true;
+      HasX86_64 = true;
   }
+  assert(!Is64Bit || HasX86_64);
   DOUT << "Subtarget features: SSELevel " << X86SSELevel
        << ", 3DNowLevel " << X863DNowLevel
        << ", 64bit " << HasX86_64 << "\n";