[FFmpeg-cvslog] r12188 - in trunk/libavcodec: armv4l/dsputil_iwmmxt.c dsputil.h i386/cputest.c i386/dsputil_mmx.c

Reimar Döffinger Reimar.Doeffinger
Sun Feb 24 20:46:05 CET 2008


On Sun, Feb 24, 2008 at 02:35:16PM -0500, Rich Felker wrote:
> On Sun, Feb 24, 2008 at 03:46:23PM +0100, reimar wrote:
> > Author: reimar
> > Date: Sun Feb 24 15:46:22 2008
> > New Revision: 12188
> > 
> > Log:
> > __asm __volatile -> asm volatile, improves code consistency and works
> > (as far as that is possible) with the Sun C compiler.
> 
> Indeed, this is much saner. __asm is reserved for the implementation
> so using it is rather inadvisible. asm is reserved for the
> application, so at worst, if the implementation doesn't support asm
> but does support __asm, then just use -Dasm=__asm and be happy. The
> other way around is not valid, though.

Well, there is still __asm__ __volatile__, but since those seem to not
cause any problems yet I'll leave them...




More information about the ffmpeg-cvslog mailing list