[PATCH] D158348: [clang][X86] Add __cpuidex function to cpuid.h

2023-08-22 Thread Aaron Ballman via Phabricator via cfe-commits
aaron.ballman added a subscriber: eandrews. aaron.ballman added a comment. @eandrews tested this patch out internally at Intel and it seems to work well for us (didn't break the internal tests that we had troubles with before). I think the changes here look reasonable, but I leave it to the

[PATCH] D158348: [clang][X86] Add __cpuidex function to cpuid.h

2023-08-19 Thread Aiden Grossman via Phabricator via cfe-commits
aidengrossman added reviewers: aaron.ballman, glandium, mstorsjo, craig.topper. aidengrossman added a comment. This is an updated version of https://reviews.llvm.org/D150646 that uses `__has_builtin` instead of checking for a definition of `_MSC_EXTENSIONS`. This fixes the cases that I have

[PATCH] D158348: [clang][X86] Add __cpuidex function to cpuid.h

2023-08-19 Thread Aiden Grossman via Phabricator via cfe-commits
aidengrossman created this revision. Herald added a project: All. aidengrossman requested review of this revision. Herald added a reviewer: jdoerfert. Herald added subscribers: cfe-commits, jplehr, sstefan1. Herald added a project: clang. MSVC has a __cpuidex function implemented to call the