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

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Fri Nov 13 10:24:57 PST 2015


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

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>

More information about the webkit-unassigned mailing list