[FFmpeg-devel] [RFC] replace some static with asm_visibility or so

Reimar Döffinger Reimar.Doeffinger
Sun Jan 27 16:06:32 CET 2008


Hello,
On Sun, Jan 27, 2008 at 04:59:28PM +0200, Uoti Urpala wrote:
> On Sun, 2008-01-27 at 15:53 +0100, Reimar D?ffinger wrote:
> > Hello,8 at 03:10:39PM +0100, Reimar D?ffinger wrote:
> > > as you may know current code does not compile with ICC and assembler due
> > > to it optimizing away all the constants.
> 
> > On second though, something like
> > #define DECLARE_ASM_CONST(n,t,v) static const t v attribute_used __attribute__ ((aligned (n)))
> > might work out nicer (less clutter).
> > Also makes sure nobody uses something global, non-const from asm code without
> > thinking.
> > Opinions, alternatives?
> 
> The "right" way to fix it would of course be to use proper asm
> arguments.

I tried it once, I am not going to try this again, if that way is chosen
someone else will have to do it.




More information about the ffmpeg-devel mailing list