From ca19564eec76831b2888c62d619b8837586d8a8a Mon Sep 17 00:00:00 2001 From: Thom Johansen Date: Sun, 12 Jun 2005 12:53:10 +0000 Subject: Enabled minor Tremor opt. git-svn-id: svn://svn.rockbox.org/rockbox/trunk@6688 a1c6a512-1295-4272-9138-f99709370657 --- apps/codecs/Tremor/asm_mcf5249.h | 11 ++++++----- 1 file changed, 6 insertions(+), 5 deletions(-) (limited to 'apps') diff --git a/apps/codecs/Tremor/asm_mcf5249.h b/apps/codecs/Tremor/asm_mcf5249.h index 9844cc05a4..a60c6ce2ad 100644 --- a/apps/codecs/Tremor/asm_mcf5249.h +++ b/apps/codecs/Tremor/asm_mcf5249.h @@ -110,14 +110,16 @@ void XNPROD31(ogg_int32_t a, ogg_int32_t b, } -#if 1 +#if 0 /* canonical definition */ #define XPROD32(_a, _b, _t, _v, _x, _y) \ { (_x)=MULT32(_a,_t)+MULT32(_b,_v); \ (_y)=MULT32(_b,_t)-MULT32(_a,_v); } -#else -/* Thom Johansen's suggestion; this could loose the LSB by overflow; - Does it matter in practice? */ +#endif + +/* this could lose the LSB by overflow, but i don't think it'll ever happen. + if anyone think they can hear a bug caused by this, please try the above + version. */ #define XPROD32(_a, _b, _t, _v, _x, _y) \ asm volatile ("mac.l %[a], %[t], %%acc0;" \ "mac.l %[b], %[v], %%acc0;" \ @@ -131,7 +133,6 @@ void XNPROD31(ogg_int32_t a, ogg_int32_t b, : [a] "r" (_a), [b] "r" (_b), \ [t] "r" (_t), [v] "r" (_v) \ : "cc"); -#endif /* asm versions of vector operations for block.c, window.c */ -- cgit v1.2.3