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

[bugzilla-daemon at webkit.org]

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

           Summary: Add RVCT compiler optimization flag Otime
           Product: WebKit
           Version: 528+ (Nightly build)
          Platform: S60 Hardware
        OS/Version: S60 3rd edition
            Status: UNCONFIRMED
          Severity: Normal
          Priority: P2
         Component: WebCore JavaScript
        AssignedTo: webkit-unassigned at lists.webkit.org
        ReportedBy: norbert.leser at nokia.com
                CC: yongjun.zhang at nokia.com, hausmann at webkit.org,
                    laszlo.1.gombos at nokia.com, siddharth.mathur at nokia.com,
                    volodimir.burlik at nokia.com



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

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #45565|                            |review+
               Flag|                            |


Created an attachment (id=45565)
 --> (https://bugs.webkit.org/attachment.cgi?id=45565)
Patch for rvct -Otime optimization

Adding the RVCT compiler (ARM target) optimization flags "-Otime -O3" (Default
settings are -Ospace and -O2) significantly increases performance of local JS
processing (dramatically better results of V8 and Sunspider tests, for
instance). The trade-offs on runtime memory consumption are relatively small,
though, text size of dll increases by about 1MB.

This compiler flag is set conditionally, for symbian platform.

The compiler chokes on , however. It appears that the compiler with -Otime
optimization flag tries to optimize out inline new'ed pointers that are passed
as arguments.
Proposed patch assigns new'ed pointer explicitly outside function call. This
should not cause any regression on other platforms.
Alternative approach would be to work around this case by using "#pragma
Ospace", conditionally for COMPILER(RVCT), but that would add an otherwise
unneeded #define.

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