[MPlayer-dev-eng] [PATCH] Make mp3lib SIMD optimizations work on AMD64, Part 5

Guillaume Poirier gpoirier at mplayerhq.hu
Fri May 25 14:06:11 CEST 2007


Hi,

Reimar Döffinger wrote:
> Hello,
> On Fri, May 25, 2007 at 02:14:49PM +0800, Zuxy Meng wrote:
>>  We've reached Part 5. This is again a big diff, replacing many 'long'
>>  to 'int', and several 'real' to 'short'. Among those, many are
>>  necessary because under LP64 environment, sizeof(long) != sizeof(int)
>>  and sizeof(long) != sizeof(float); others are optimizations simply
>>  because we really don't need such a long type, or the declaration
>>  didn't match its use (declared as float, used as short).
> 
> These are separate issues and should be fixed in different patches. Also
> if you changed long to int because the code assumes 32 bits, it would
> make more sense to change it to int32_t. Replacing long with int without
> thinking can make the code even slower on 64 bit systems.

[..]

> 
> This is unrelated and also completely useless since a static function
> that is used in only one place will be inlined anyway.
> In addition it is wrong since "static void make_decode_tables()" !=
> "static void make_decode_tables(void)"


Thanks for your quality review Reimar! I realize I'm quite a lousy
reviewer, but I'm trying to do my best! :-(

Guillaume



More information about the MPlayer-dev-eng mailing list