[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 #17 from Michael Catanzaro <mcatanzaro at igalia.com> ---
Comment on attachment 265479
  --> https://bugs.webkit.org/attachment.cgi?id=265479
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....

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

> 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. ;)

-- 
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/20151113/3ff95fa3/attachment.html>