rnk added inline comments.
================ Comment at: lib/Headers/intrin.h:196 + */ void __cdecl _invpcid(unsigned int, void *); +#endif ---------------- GBuella wrote: > GBuella wrote: > > rnk wrote: > > > craig.topper wrote: > > > > @rnk, what's the right thing to do here? > > > What problems does this redeclaration cause? > > Now that I think about it, none. > Also, I think this could be added as TARGET_HEADER_BUILTIN..."intrin.h", > ALL_MS_LANGUAGES into BuiltinsX86.def > And removed from here. > Right? It could be, but then you'd need to implement it as a builtin instead of having invpcidintrin.h, right? Repository: rC Clang https://reviews.llvm.org/D47142 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits