[Webkit-unassigned] [Bug 32988] Add RVCT compiler optimization flag Otime

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=32988


Norbert Leser <norbert.leser at nokia.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #45565|0                           |1
        is obsolete|                            |




--- Comment #7 from Norbert Leser <norbert.leser at nokia.com>  2009-12-31 05:40:13 PST ---
Created an attachment (id=45707)
 --> (https://bugs.webkit.org/attachment.cgi?id=45707)
Updated patch for bug #32988

According to suggestions, the updated patch compiles JavaScriptCore related
code only with -Otime -O3 compiler options.

Since we are dealing with one big library, it is hard to separate JSC from the
rest, specifically since there is this name clash of the common config.h file
(i.e., there is no common config header just for JSC). I opted for adding the
pragma statement for the optimization flags into runtime/structure.h which
appears
to be one of the most common header files used by JSC code.

I still get pretty good performance results (though, about 8% less than with
full optimization of all webkit code).
The space increase of the dll (in my comparative builds) is from 4,178 kB to
4,561 kB (the full optimization generates dll with 4,800+ kB).
Code size increase via elftran show from 006edafc to 007d9d8c.

I still need to get data in from QA regression tests with more comparison of
startup time etc.

I did not turn the review flag back on yet - before I get more feedback.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.