davezarzycki added a comment.
I think this change is incomplete. The following clang test fails on my SKX
machine: `Preprocessor/predefined-arch-macros.c`
Line 895: `// CHECK_SKX_M32: #define __SGX__ 1`
Repository:
rL LLVM
https://reviews.llvm.org/D45058
_
This revision was automatically updated to reflect the committed changes.
Closed by commit rL329701: [X86] Disable SGX for Skylake Server (authored by
GBuella, committed by ).
Herald added a subscriber: llvm-commits.
Repository:
rL LLVM
https://reviews.llvm.org/D45058
Files:
cfe/trunk/lib/B
This revision was automatically updated to reflect the committed changes.
Closed by commit rC329701: [X86] Disable SGX for Skylake Server (authored by
GBuella, committed by ).
Changed prior to commit:
https://reviews.llvm.org/D45058?vs=140302&id=141839#toc
Repository:
rL LLVM
https://review
craig.topper accepted this revision.
craig.topper added a comment.
This revision is now accepted and ready to land.
LGTM
Repository:
rC Clang
https://reviews.llvm.org/D45058
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llv
GBuella created this revision.
GBuella added reviewers: craig.topper, zvi, echristo.
Herald added a subscriber: cfe-commits.
Repository:
rC Clang
https://reviews.llvm.org/D45058
Files:
lib/Basic/Targets/X86.cpp
Index: lib/Basic/Targets/X86.cpp
==