[Webkit-unassigned] [Bug 142128] [GTK] Use FTL by default when LLVM 3.7 is available

[bugzilla-daemon at webkit.org]

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

--- Comment #18 from Carlos Garcia Campos <cgarcia at igalia.com> ---
(In reply to comment #17)
> Comment on attachment 265479 [details]
> Updated patch
> 
> View in context:
> https://bugs.webkit.org/attachment.cgi?id=265479&action=review
> 
> Note that Apple wrote b3 to replace LLVM in the FTL, so this dependency is
> going to go away in the near-ish future....

That's still work in progress anyway.

> > Source/JavaScriptCore/PlatformGTK.cmake:42
> > +    install(TARGETS llvmForJSC
> 
> Hm, what is up with llvmForJSC? Why are we loading this as a shared library
> at runtime? I don't see why we need a shared library; surely we should be
> able to just add those files to the javascriptcoregtk archive?

https://trac.webkit.org/changeset/157260

> > Source/cmake/OptionsGTK.cmake:136
> > +WEBKIT_OPTION_DEFAULT_PORT_VALUE(ENABLE_FTL_JIT PUBLIC ${ENABLE_FTL_DEFAULT})
> 
> Move this up, since the rest are alphabetized. ;)

Ok.

-- 
You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.webkit.org/pipermail/webkit-unassigned/attachments/20151116/fbf4606d/attachment.html>