Bug List: (This bug is not in your last search results)   Show last search results      Search page      Enter new bug
Bug#: 87539
Alias:
Product:
Component:
Status: RESOLVED
Resolution: FIXED
Assigned To: Gentoo Sound Team <sound@gentoo.org>
Hardware:
OS:
Version:
Priority:
Severity:
Reporter: Simon Strandman <simon.strandman@telia.com>
Add CC:
CC:
URL:
Summary:
Status Whiteboard:
Keywords:

Filename Description Type Creator Created Size Actions
x86_64_CFLAGS.patch Patch. patch Simon Strandman 2005-04-01 03:50 0000 854 bytes Details | Diff
Create a New Attachment (proposed patch, testcase, etc.) View All

Bug 87539 depends on: Show dependency tree
Bug 87539 blocks:
Votes: 0    Show votes for this bug    Vote for this bug

Additional Comments: (this is where you put emerge --info)


Not eligible to see or edit group visibility for this bug.






View Bug Activity   |   Format For Printing   |   XML   |   Clone This Bug


Description:   Opened: 2005-04-01 03:49 0000
Mpg123 doesn't use CFLAGS from make.conf on amd64. I have made a patch to fix
that. I have also tries to compile it with a lot of different CFLAGS
combinations and it seems stable even with insane optimizations.

Reproducible: Always
Steps to Reproduce:

------- Comment #1 From Simon Strandman 2005-04-01 03:50:12 0000 -------
Created an attachment (id=55023) [details]
Patch.

------- Comment #2 From Danny van Dyk (RETIRED) 2005-07-15 23:44:15 0000 -------
Has nothing to do with amd64.

------- Comment #3 From Simon Strandman 2005-10-15 12:18:11 0000 -------
Anything happening here?

Here are some results:

Before applying my patch (compiling unoptimized):

[12:18] Decoding of foo.mp3 finished.

real    0m14.533s
user    0m14.133s
sys     0m0.358s

After applying my patch:

[12:18] Decoding of foo.mp3 finished.

real    0m3.942s
user    0m3.628s
sys     0m0.271s

As you can see the patched mpg123 is more than three times faster. So please
apply it. :)

------- Comment #4 From Diego E. 'Flameeyes' Pettenò 2005-10-17 04:54:25 0000 -------
Thanks, fixed in -r10 in CVS. 

Bug List: (This bug is not in your last search results)   Show last search results      Search page      Enter new bug