Bug 118560

Summary:

DFG should support intrinsic for Math.pow().

Product:

WebKit

Reporter:

Vladislav <vlad>

Component:

JavaScriptCore

Assignee:

Nobody <webkit-unassigned>

Status:

RESOLVED DUPLICATE

Severity:

Enhancement

CC:

benjamin, bfulgham, fpizlo, ggaren, mhahnenberg, msaboff, oliver

Priority:

Version:

528+ (Nightly build)

Hardware:

All

OS:

All

Attachments:

Description	Flags
The patch	fpizlo: review+

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 ***