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 run automatically precommit around here, now I know I must always check them manually... sorry https://reviews.llvm.org/D45488 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
