[PATCH] D157115: Revert "[clang][X86] Add __cpuidex function to cpuid.h"

Aaron Ballman via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Aug 4 12:03:28 PDT 2023


aaron.ballman added a comment.

Thank you for this! The revert LGTM, but please wait for @mstorsjo to confirm this won't cause problems for MinGW folks.

In terms of solving the underlying issue with aux triple so that we can re-land this, I went digging to see if there's an existing issue tracking this and I could not find one. However, related to the functionality in this patch, I did see: https://github.com/llvm/llvm-project/issues/60383. I filed https://github.com/llvm/llvm-project/issues/64436 to track the aux-triple issue; hopefully we can find someone with the bandwidth to look into that issue (I'll see if I can get someone at Intel to look into it, but if someone else wanted to tackle it first, that would be great).


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D157115/new/

https://reviews.llvm.org/D157115



More information about the cfe-commits mailing list