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