[PATCH] D29464: [MinGWToolChain] Don't use GCC headers on Win32

Reid Kleckner via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Feb 14 10:04:30 PST 2017


rnk accepted this revision.
rnk added a comment.

So far as I can tell, Clang on Linux supports none of the headers under discussion here, and that's been OK for some time. None of the headers in question appear to actually work today, so I think we should remove them from the search path.

Regarding OpenMP, we should do whatever Intel is doing to make OpenMP work in Clang, which doesn't rely on omp.h from libgcc. quadmath.h seems like something that should be part of compiler-rt if we want to support it. If users want to use quadmath functions from libgcc, then I think it's fair for them to have to do extra work to pull that header out so that they don't accidentally pull in conflicting GCC headers like immintrin.h and limits.h.

@yaron.keren does that seem reasonable?


https://reviews.llvm.org/D29464





More information about the cfe-commits mailing list