On 07/21/13 17:02, Erik van Pienbroek wrote: > Erik van Pienbroek schreef op zo 21-07-2013 om 14:49 [+0200]: >> So now I think it's up to us to come up with the most proper fix which >> we can then try to get upstreamed. > Attached is the patch I came up with to fix the build issue. It is > basically method 2 in dw's original mail. > > The header in question already contained an #include <intrin.h>, but it > was only part of an #ifdef _MSC_VER section (which doesn't get triggered > for mingw-w64). I've changed this #ifdef clause so that it also includes > intrin.h when the mingw-w64 compiler is used. > > With this patch I've done the following build tests: > > using an old mingw-w64 snapshot (20121016): success! > using a recent mingw-w64 snapshot (20130713, r5949): success! > using a bleeding edge mingw-w64 snapshot (20130721, r5969): success! > > The qpid-cpp build failure is also resolved with this patch as well. > > Unless somebody objects I think this patch is good enough to be proposed > upstream.
Looks good to me. Thanks, Jacek ------------------------------------------------------------------------------ See everything from the browser to the database with AppDynamics Get end-to-end visibility with application monitoring from AppDynamics Isolate bottlenecks and diagnose root cause in seconds. Start your free trial of AppDynamics Pro today! http://pubads.g.doubleclick.net/gampad/clk?id=48808831&iu=/4140/ostg.clktrk _______________________________________________ Mingw-w64-public mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/mingw-w64-public
