[Webkit-unassigned] [Bug 29946] [Qt] Port YARR JIT for Symbian (implement ExecutableAllocator)

[bugzilla-daemon at webkit.org]

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


Simon Hausmann <hausmann at webkit.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #41213|review?                     |review+
               Flag|                            |




--- Comment #5 from Simon Hausmann <hausmann at webkit.org>  2009-10-15 10:38:32 PDT ---
(From update of attachment 41213)

> +#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!

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