[FFmpeg-devel] [PATCH] remove MSVC cruft

Rich Felker dalias
Sun Feb 10 05:01:47 CET 2008


On Sat, Feb 09, 2008 at 05:54:16PM +0100, Michael Niedermayer wrote:
> > Just as well for me, but I very much dislike the current way of making
> > the least supported variant the default case.
> > Though what I do not like much about your suggestion is that the default
> > case will now probably create broken code with only a warning...
> > Given that Intel and Solaris compilers both (partially?) support the gcc syntax, I'd
> > suggest to make the gcc case the default one (assuming nobody wants to
> > make up a proper configure check).
> 
> Iam fine with that as well.

I'm against having any gcc-specific code in the default build when the
ability to compile gnu c (e.g. inline asm) has not been ascertained.
There's no use of these alignment directives if asm is disabled,
anyway, and only sse code will actually crash (otherwise performance
is just bad).

Rich




More information about the ffmpeg-devel mailing list