Notice: A non well formed numeric value encountered in /sites/rockbox.org/flyspray/includes/class.flyspray.php on line 96 Notice: A non well formed numeric value encountered in /sites/rockbox.org/flyspray/includes/class.flyspray.php on line 96 Notice: A non well formed numeric value encountered in /sites/rockbox.org/flyspray/includes/class.flyspray.php on line 96 Deprecated: Function create_function() is deprecated in /sites/rockbox.org/flyspray/includes/class.flyspray.php on line 104 Deprecated: The each() function is deprecated. This message will be suppressed on further calls in /sites/rockbox.org/flyspray/adodb/adodb.inc.php on line 845 Deprecated: Function create_function() is deprecated in /sites/rockbox.org/flyspray/includes/class.user.php on line 111 FS#5707 : Compiler optimisations for libmad on iPod 3G

Rockbox

Tasklist

FS#5707 - Compiler optimisations for libmad on iPod 3G

Attached to Project: Rockbox
Opened by Daniel Ankers (dan_a) - Sunday, 23 July 2006, 23:19 GMT
Task Type Patches
Category Codecs
Status Closed
Assigned To No-one
Operating System iPod 3G
Severity Low
Priority Normal
Reported Version
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 0
Private No

Details

I have found that there is a large benefit on the iPod 3g if libmad is compiled using -O instead of -O2.
This patch adds an extra environment variable, EXTRAMADOPTS, to tools/configure, and adds it after the normal libmad compiler options to allow different settings on a per-port basis.

It may be worth testing the effects of changing this option in other ports.
This task depends upon

Closed by  Dave Chapman (linuxstb)
Monday, 24 July 2006, 19:33 GMT
Reason for closing:  Accepted
Additional comments about closing:  This also gave a speedup on 4g and later ipods, so I amended your patch accordingly before committing. Thanks!
Comment by Daniel Ankers (dan_a) - Monday, 24 July 2006, 00:00 GMT
Following discussion with linuxstb here is an updated version of the patch.
It does not add the environment variable, and instead uses a conditional in the libmad Makefile.

Loading...