[MPlayer-dev-eng] altivec patch 4/5: RGB/BGR confusion

[Guillaume POIRIER]

Hi Alan,

You seem to be interested in improving Altivec support.

If you have a chance, I think it would be great if you could see what
may be wrong with 'hadamard8_diff16_altivec' and
'hadamard8_diff8x8_altivec' which prevent them from working correctly
with GCC 3.3 and GCC 3.4...

Those work alright on OSX, but fail with FSF GCC (another proof that
intrinsic code != asm code as it relies on some compiler's
intelligence)

Rather that fixing them, it may be interesting to see if GCC4.0 or 4.1
can, at last, generate good code from these intrinsics.

from dsputil_ppc.c:295

#ifdef CONFIG_DARWIN // ATM gcc-3.3 and gcc-3.4 fail to compile these
in linux...
        c->hadamard8_diff[0] = hadamard8_diff16_altivec;
        c->hadamard8_diff[1] = hadamard8_diff8x8_altivec;
#endif

hadamard* functions are in dsputil_altivec.c

Guillaume
--
Just because code is syntactically "valid" GNU C doesn't mean gcc can
always compile it.
  Steven Bosscher - 2005-01-01
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=11203#c14
---
MPlayer's doc isn't up-to-date. Visit my updated mirror here:
http://tuxrip.free.fr//MPlayer-DOCS-HTML/en/
http://tuxrip.free.fr//MPlayer-DOCS-HTML/fr/