[FFmpeg-cvslog] r11889 - trunk/libavutil/mem.h

[Reimar Döffinger]

Hello,
On Mon, Feb 11, 2008 at 01:35:24PM -0500, Rich Felker wrote:
> On Mon, Feb 11, 2008 at 07:17:18PM +0100, Reimar D?ffinger wrote:
> > On Mon, Feb 11, 2008 at 12:51:08PM -0500, Rich Felker wrote:
> > > On Mon, Feb 11, 2008 at 11:45:38AM +0100, Reimar D?ffinger wrote:
> > > > Huh? With my suggestion the gcc variant is the fallback case...
> > > 
> > > The fallback case needs to be C, not "GNU C" or "MS Visual C++" or
> > > whatever they call it now. A plain C build has no need for alignment
> > > beyond what the compiler provides, nor any need for asm constants
> > > since there's no asm.
> > 
> > I'll agree at once if you change configure so this is the case.
> > But the fallback case being "plain C but with asm that is sure to crash
> > because of missing alignment" is idiocy.
> 
> Umm, does configure really detect the support of asm without actually
> checking for it?? Somehow I doubt this. It should at least look for
> the __GNUC stuff or attempt to compile a file with inline asm..

And how does that help in the not gnu but asm supported case (currently
that would be ICC and Solaris compilers, maybe some other as well?)