Bug 117147 - [sh4] Add floating point absolute function support in baseline JIT
Summary: [sh4] Add floating point absolute function support in baseline JIT
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: JavaScriptCore (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2013-06-03 07:21 PDT by Julien Brianceau
Modified: 2013-06-04 09:59 PDT (History)
5 users (show)

See Also:


Attachments
Add floating point absolute function support in sh4 baseline JIT (2.97 KB, patch)
2013-06-03 07:30 PDT, Julien Brianceau
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Julien Brianceau 2013-06-03 07:21:24 PDT
Add floating point absolute function support in sh4 baseline JIT.
Comment 1 Julien Brianceau 2013-06-03 07:30:06 PDT
Created attachment 203587 [details]
Add floating point absolute function support in sh4 baseline JIT
Comment 2 WebKit Commit Bot 2013-06-03 07:31:31 PDT
Attachment 203587 [details] did not pass style-queue:

Failed to run "['Tools/Scripts/check-webkit-style', '--diff-files', u'Source/JavaScriptCore/ChangeLog', u'Source/JavaScriptCore/assembler/MacroAssemblerSH4.h', u'Source/JavaScriptCore/assembler/SH4Assembler.h']" exit_code: 1
Source/JavaScriptCore/assembler/SH4Assembler.h:181:  enum members should use InterCaps with an initial capital letter.  [readability/enum_casing] [4]
Total errors found: 1 in 3 files


If any of these errors are false positives, please file a bug against check-webkit-style.
Comment 3 Julien Brianceau 2013-06-03 07:44:20 PDT
I've tested this patch on r151092 and I didn't see regressions when running
- Tools/Scripts/run-javascriptcore-tests
- Tools/Scripts/run-fast-jsc
- SunSpider 1.0


I used the following dumb test to measure the performance impact of this patch:

    var result = 1.123;
    for (var i = 0; i < 5000000; ++i) {
        result = result + Math.abs((i / -(i+1)) + (i * 1.1));
    }


Using r151092 jsc without patch, I get:
    real    0m 6.13s
    user    0m 6.00s
    sys     0m 0.10s


Using r151092 jsc with patch, I get about 4.3% better:
    real    0m 5.57s
    user    0m 5.45s
    sys     0m 0.10s
Comment 4 Geoffrey Garen 2013-06-04 09:37:35 PDT
Comment on attachment 203587 [details]
Add floating point absolute function support in sh4 baseline JIT

r=me
Comment 5 WebKit Commit Bot 2013-06-04 09:59:01 PDT
Comment on attachment 203587 [details]
Add floating point absolute function support in sh4 baseline JIT

Clearing flags on attachment: 203587

Committed r151174: <http://trac.webkit.org/changeset/151174>
Comment 6 WebKit Commit Bot 2013-06-04 09:59:03 PDT
All reviewed patches have been landed.  Closing bug.