Changed CFLAGS for a speed increase.

Bug #6003 reported by Yagisan
6
Affects Status Importance Assigned to Milestone
transcode (Ubuntu)
Invalid
Medium
MOTU Media Team

Bug Description

Changed to CFLAGS from "-O2" to "-O2 -fweb -ffast-math -funswitch-loops -fgcse-after-reload -fomit-frame-pointer" for a small speed increase. Tested on amd64 and k7 machines.

Revision history for this message
Yagisan (yagisan) wrote : Patch to change the CFLAGS

This patch changes the CFLAGS in debian/rules to "-O2 -fweb -ffast-math -funswitch-loops -fgcse-after-reload -fomit-frame-pointer"

Changed in transcode:
assignee: nobody → motumedia
Yagisan (yagisan)
Changed in transcode:
assignee: motumedia → slomo
Sebastian Dröge (slomo)
Changed in transcode:
assignee: slomo → motumedia
Revision history for this message
Matthias Klose (doko) wrote :

- such changes have to be tested on non k7/amd64 hardware as well.
- "small" is a bit useless

Revision history for this message
Reinhard Tartler (siretart) wrote :

Yagisan: could you please provide numbers to the speed increase and provide perhaps a small testcase how to get these numbers? you could place this information somewhere in debian/ with a small comment how to use them.

If the speed increase is reproducible and significant, we could enable them for the tested architecture only. You cannot persume that the cflags you provide don't do any harm on other architectures.

Changed in transcode:
status: Unconfirmed → Needs Info
Revision history for this message
Yagisan (yagisan) wrote :

Sure. When I finish getting my boxes rebuilt. IIRC it was a 1-5 fps increase.
All I have to test on here is an athlon64 2Ghz, duron 850Mhz, k6/2 333Mhz, and a pentium 2 233Mhz, so will obviously want some ppc, and newer intel boxes to test.

Revision history for this message
Vassilis Pandis (pandisv) wrote :

Can you please provide the information requested by Reinhard? Thanks.

Revision history for this message
Vassilis Pandis (pandisv) wrote :

We are closing this bug report as it lacks the information, described in the previous comments, we need to investigate the problem further. However, please reopen it if you can give us the missing information and don't hesitate to submit bug reports in the future.

Changed in transcode:
status: Needs Info → Rejected
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.