This revision was automatically updated to reflect the committed changes.
Closed by commit rL329710: [X86] Disable SGX for Skylake Server - CPP test
(authored by GBuella, committed by ).
Herald added a subscriber: llvm-commits.
Changed prior to commit:
https://reviews.llvm.org/D45488?vs=141853&
GBuella added a comment.
In https://reviews.llvm.org/D45488#1063067, @davezarzycki wrote:
> I don't think this change is correct. To the best of my knowledge, SKL does
> support SGX but SKX does not.
llvm-lit test/Preprocessor/predefined-arch-macros.c passes now.
I didn't know tests are not ru
GBuella updated this revision to Diff 141853.
https://reviews.llvm.org/D45488
Files:
test/Preprocessor/predefined-arch-macros.c
Index: test/Preprocessor/predefined-arch-macros.c
===
--- test/Preprocessor/predefined-arch-macros.c
davezarzycki added a comment.
I don't think this change is correct. To the best of my knowledge, SKL does
support SGX but SKX does not.
Repository:
rC Clang
https://reviews.llvm.org/D45488
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
GBuella created this revision.
GBuella added reviewers: craig.topper, davezarzycki.
Herald added a subscriber: cfe-commits.
Fix test case - corresponding to r329701
Repository:
rC Clang
https://reviews.llvm.org/D45488
Files:
test/Preprocessor/predefined-arch-macros.c
Index: test/Preproce