[PATCH] D99465: [Clang][X86] Mark some VAES builtins as conditionally defined

2021-03-28 Thread Craig Topper via Phabricator via cfe-commits
craig.topper added a comment. I think I've fixed this in 3fb40ce167ff5f05afadf8f525ff9e17350d6d7f . We need to use the preprocessor define that says avx512fintrin.h was included rather than the feature define. We already did a

[PATCH] D99465: [Clang][X86] Mark some VAES builtins as conditionally defined

2021-03-28 Thread LemonBoy via Phabricator via cfe-commits
LemonBoy added a comment. In D99465#2654633 , @craig.topper wrote: > We can't use preprocessor defines in the intrinsic header files. It prevents > being able to use the intrinsics with attribute(target("avx512f,vaes")). > What's preventing __m512i from

[PATCH] D99465: [Clang][X86] Mark some VAES builtins as conditionally defined

2021-03-28 Thread Craig Topper via Phabricator via cfe-commits
craig.topper added a comment. We can't use preprocessor defines in the intrinsic header files. It prevents being able to use the intrinsics with attribute(target("avx512f,vaes")). What's preventing __m512i from being defined? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION h

[PATCH] D99465: [Clang][X86] Mark some VAES builtins as conditionally defined

2021-03-28 Thread LemonBoy via Phabricator via cfe-commits
LemonBoy created this revision. LemonBoy added a reviewer: craig.topper. LemonBoy requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. AMD Zen3 architecture supports the VAES instruction set but not the 512bit variants, add a feature gate on __