Thank you for your explaination.
Patch is ok.
Thanks
Kai
Am 13.07.2013 20:03 schrieb "dw" <[email protected]>:
>
> > Point 3 should not force none-inline version. Please expain why you
> > think that is required.
>
> While I removed the inline asm from these 3 routines, the routines
> themselves are still __CRT_INLINE. And the routines they call are
> either __CRT_INLINE or __MINGW_INTRIN_INLINE. If you examine the output
> generated, you'll see these routines still only generate a single line
> of asm code. Hard to get more efficient than that.
>
> If I weren't changing them to call existing routines, I'd still have to
> change the asm. As written, they don't support -masm=intel (one of my
> goals). Rather than duplicate the inline asm from elsewhere, calling
> existing routines seemed the more sensible plan.
>
> I might also point out that #3 only affects x86 code. The x64 code
> already does it this way.
>
> I'll wait for your final approval before committing.
>
> dw
>
>
> ------------------------------------------------------------------------------
> 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
>
------------------------------------------------------------------------------
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
