[MPlayer-dev-eng] icc benchmark

[Michael Niedermayer]

Hi

On Mon, Jun 11, 2007 at 10:54:09PM +0000, Carl Eugen Hoyos wrote:
> Hi!
> 
> Uoti Urpala <uoti.urpala <at> pp1.inet.fi> writes:
> 
> > > I'd be more than happy to hear of a simpler alternative.
> > 
> > I think the only non-hacky/ugly way that would also be "guaranteed" to
> > work (as long as the compilers actually work...) would be to use asm
> > constraints. I don't know if there's still compiler brokenness which
> > would prevent this from working with icc 10.0; at least some tests I
> > made seemed to indicate it working correctly.
> > 
> > Attached is an example change for liba52/imdct.c (chosen just because it
> > was the last file among initial icc error messages). The READ_ARRAY
> > macro would obviously be defined in a header instead of per file. This
> > version seems to work right with both gcc and icc.
> 
> I could have bet on it: It doesn't work with gcc-2.95.3

the problem here is that the odds are 1:100 or so that uoti will post a
patch which breaks gcc 2.95
so maybe you rather should bet that he posts one which doesnt and hope
in human error ...


> 
> Except for namespace polution: Is there another disadvantage with global
> variables?

it doesnt break gcc 2.95 ...

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

No snowflake in an avalanche ever feels responsible. -- Voltaire
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/mplayer-dev-eng/attachments/20070612/e303c7d8/attachment.pgp>