r262200 - [X86] Enabling xsave should not enable AVX. I seem to have done this, but I don't know why.

Craig Topper via cfe-commits cfe-commits at lists.llvm.org
Sun Feb 28 22:51:34 PST 2016


Author: ctopper
Date: Mon Feb 29 00:51:34 2016
New Revision: 262200

URL: http://llvm.org/viewvc/llvm-project?rev=262200&view=rev
Log:
[X86] Enabling xsave should not enable AVX. I seem to have done this, but I don't know why.

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=262200&r1=262199&r2=262200&view=diff
==============================================================================
--- cfe/trunk/lib/Basic/Targets.cpp (original)
+++ cfe/trunk/lib/Basic/Targets.cpp Mon Feb 29 00:51:34 2016
@@ -3018,15 +3018,11 @@ void X86TargetInfo::setFeatureEnabledImp
     else
       setSSELevel(Features, SSE41, Enabled);
   } else if (Name == "xsave") {
-    if (Enabled)
-      setSSELevel(Features, AVX, Enabled);
-    else
+    if (!Enabled)
       Features["xsaveopt"] = false;
   } else if (Name == "xsaveopt" || Name == "xsavec" || Name == "xsaves") {
-    if (Enabled) {
+    if (Enabled)
       Features["xsave"] = true;
-      setSSELevel(Features, AVX, Enabled);
-    }
   }
 }
 




More information about the cfe-commits mailing list