Bug 118560 - DFG should support intrinsic for Math.pow().
Summary: DFG should support intrinsic for Math.pow().
Status: RESOLVED DUPLICATE of bug 141540
Alias: None
Product: WebKit
Classification: Unclassified
Component: JavaScriptCore (show other bugs)
Version: 528+ (Nightly build)
Hardware: All All
: P2 Enhancement
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2013-07-11 05:13 PDT by Vladislav
Modified: 2015-02-18 17:32 PST (History)
7 users (show)

See Also:


Attachments
The patch (25.73 KB, patch)
2013-07-11 05:49 PDT, Vladislav
fpizlo: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Vladislav 2013-07-11 05:13:41 PDT
DFG intrinsic as opposed to old JIT intrinsic can use speculative information about types. Also it allows to process constants at compile time.
Concrete assembler for ARMv7 is used for better performance (MacroAssembler and calls to C functions for other architectures).
Comment 1 Vladislav 2013-07-11 05:49:36 PDT
Created attachment 206452 [details]
The patch
Comment 2 Vladislav 2013-07-26 06:34:46 PDT
It gives about 12% speedup on SunSpider's math-partial-sums on ARMv7 Linux while neutral on other tests.
Comment 3 Brent Fulgham 2013-10-30 10:26:10 PDT
This seems like a good idea. Is there any reason it's not being merged?
Comment 4 Filip Pizlo 2013-10-30 11:31:11 PDT
Comment on attachment 206452 [details]
The patch

r=me.  Can you rebase?
Comment 5 Vladislav 2013-10-31 08:01:25 PDT
(In reply to comment #4)
> (From update of attachment 206452 [details])
> r=me.  Can you rebase?

Sure.
Comment 6 Benjamin Poulain 2015-02-18 17:32:30 PST

*** This bug has been marked as a duplicate of bug 141540 ***