2012/12/22 Jacek Caban <ja...@codeweavers.com>: > On 12/22/12 8:41 PM, Ruben Van Boxem wrote: > > 2012/12/22 Jacek Caban <ja...@codeweavers.com> >> >> On 12/22/12 1:43 PM, Ruben Van Boxem wrote: >> >> 2012/12/22 Jacek Caban <ja...@codeweavers.com> >>> >>> On 12/22/12 12:23 PM, Kai Tietz wrote: >>> > Please include first the windows.h header, and then do other includes. >>> > >>> > There is nothing to fix on mingw-w64's side. It might be a >>> > gcc-intrinsic header bug, as API should be artificial but still has >>> > language-binding. AFAIR there is already a bug-report on gcc's bz >>> > about that. >>> > >>> > See Windows.cpp: line 59 ... move this include at to of include-chain. >>> > >>> >>> We already have workarounds for this kind of things in the tree, we just >>> need to extend it for this particular intrin. I will prepare a patch >>> later. >> >> >> Thanks! >> >> >> Here is a patch ready for tests: >> >> http://repo.or.cz/w/mingw-w64/jacek.git/commitdiff/5fae66d9afe5ed89f4a5f6123a5559b138ac23e1 >> I will have it tested and ready for commit tomorrow, after my cron jobs >> run with it deployed. > > > My quick testing by manually applying the patch to the previously built > toolchain shows you missed one in winnt.h: > > > Right, my simplified test case didn't catch it. We may simply remove __rdtsc > from winnt.h, since we include intrin.h there anyway. > > Jacek
Yes, and cygwin can live my it AFAICS. If tests are successful, the patch is pre-approved. Thanks, Kai ------------------------------------------------------------------------------ LogMeIn Rescue: Anywhere, Anytime Remote support for IT. Free Trial Remotely access PCs and mobile devices and provide instant support Improve your efficiency, and focus on delivering more value-add services Discover what IT Professionals Know. Rescue delivers http://p.sf.net/sfu/logmein_12329d2d _______________________________________________ Mingw-w64-public mailing list Mingw-w64-public@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/mingw-w64-public