Bug 29946 – [Qt] Port YARR JIT for Symbian (implement ExecutableAllocator)

• Status: RESOLVED
• Resolution: FIXED
• Component: JavaScriptCore

[Laszlo Gombos]

YARR JIT is currently not enabled for Symbian. This bug is to track the work of enabling it for Symbian. The scope of this bug does not include enabling JIT - that should be tracked as a separate bug.

[Laszlo Gombos]

Created attachment 40416 [details]
First try

Implements ExecutableAllocator for Symbian and turns on YARR JIT.

I did not set the review flag yet as it needs more testing; but I wanted to share the code.

[Laszlo Gombos]

Created attachment 40981 [details]
Implement ExecutableAllocator

Tested in an ARMv5 arch configuration. Increased the pageSize for ARMv5 (and less) to decrease the number of Symbian chunks.

On the Sunspider Regexp test (http://www2.webkit.org/perf/sunspider-0.9/regexp-dna.html) saw ~11 times performance improvement (from 5170 ms to 465 ms).

[Laszlo Gombos]

Created attachment 40982 [details]
Implement ExecutableAllocator - II.

Uploaded the old patch by mistake - this time upload the new patch that is up for review.

[Laszlo Gombos]

Created attachment 41213 [details]
ExecutableAllocator III.

Made it more explicit that the page_size is set to 256 Kb regardless of the actual page_size to compensate for the moving mem model.

[Simon Hausmann]

Comment on attachment 41213 [details]
ExecutableAllocator III.


> +#if ARM_ARCH_VERSION <= 5

This is really a nit-pick comment from here, but in the unlikely event of us ever introducing
the JIT for the emulator (hehehe) I guess then this should've been using PLATFORM_ARM_ARCH(5) in reverse logic, that also
verifies the presence of PLATFORM(ARM). Just a thought though, feel free to leave it as it is right now, as certainly
with the current code ENABLE(ASSEMBLER) will only be used when compiling for arm.

Otherwise the patch looks good to me!

[Laszlo Gombos]

Landed as http://trac.webkit.org/changeset/49679 - with the change suggested by Simon.