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

Aiden Grossman via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed May 17 23:05:11 PDT 2023


aidengrossman added a comment.

Thanks for notifying me of this. I've reverted it as I don't have too much time currently to thoroughly go through things and I don't want to keep you blocked. Essentially, the built-in for `__cpuidex` on Windows platforms landed in https://reviews.llvm.org/D121653 which is conflicting with this new implementation (from what I can gather). If that's the issue though, it's interesting that it doesn't seem like there are any conflicts between the `__cpuid` builtin and the one defined in `cpuid.h`. Hoping to get a fix up (relatively) soon but not sure currently exactly what it will look like. @glandium Do you have a link to a source file that ends up triggering this error in Firefox? I'd be interested in seeing the context.


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