[PATCH] D53042: [X86] Remove FeatureRTM from Skylake processor list
Craig Topper via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Tue Oct 9 17:40:07 PDT 2018
craig.topper added inline comments.
================
Comment at: lib/Basic/Targets/X86.cpp:169
setFeatureEnabledImpl(Features, "mpx", true);
if (Kind != CK_SkylakeServer) // SKX inherits all SKL features, except SGX
setFeatureEnabledImpl(Features, "sgx", true);
----------------
lebedev.ri wrote:
> > high-end desktop and server chips (I can't find any SKX without).
>
> Are you sure you didn't want to move it into an else of this `if()` ?
> This commit leaves it disabled for all processors, but can be re-enabled for specific builds with -mrtm.
So I think this was intended.
Repository:
rC Clang
https://reviews.llvm.org/D53042
More information about the cfe-commits
mailing list