[PATCH] D150646: [clang][X86] Add __cpuidex function to cpuid.h
Aiden Grossman via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Fri Aug 11 16:55:22 PDT 2023
aidengrossman added a comment.
The main issue here is that there's no way to reliably detect whether the built in is defined (can't check if a function is defined in the preprocessor, preprocessor macro isn't exposed correctly in all configurations), which ends up breaking some configurations. I wouldn't be opposed to exposing things more often, but I'm fine with the builtins being exposed under `-fms-extensions`, we just need to expose the preprocessor macro when we expose the builtins.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D150646/new/
https://reviews.llvm.org/D150646
More information about the cfe-commits
mailing list