Bug 140658 - [iOS] Only use LLVM static library arguments on 64-bit builds of libllvmForJSC.dylib
Summary: [iOS] Only use LLVM static library arguments on 64-bit builds of libllvmForJS...
Alias: None
Product: WebKit
Classification: Unclassified
Component: JavaScriptCore (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: David Kilzer (:ddkilzer)
Depends on:
Reported: 2015-01-19 20:00 PST by David Kilzer (:ddkilzer)
Modified: 2015-01-19 21:09 PST (History)
6 users (show)

See Also:

Patch v1 (1.91 KB, patch)
2015-01-19 20:08 PST, David Kilzer (:ddkilzer)
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description David Kilzer (:ddkilzer) 2015-01-19 20:00:21 PST
When building libllvmForJSC.dylib, the LLVM static library arguments (defined in Source/JavaScriptCore/Configurations/LLVMForJSC.xcconfig) are only needed for 64-bit builds since the FTL JIT is only enabled on 64-bit architectures.

This has an additional benefit of fixing the builders in the ios-ews queue.
Comment 1 David Kilzer (:ddkilzer) 2015-01-19 20:08:26 PST
Created attachment 244958 [details]
Patch v1
Comment 2 WebKit Commit Bot 2015-01-19 21:09:01 PST
Comment on attachment 244958 [details]
Patch v1

Clearing flags on attachment: 244958

Committed r178694: <http://trac.webkit.org/changeset/178694>
Comment 3 WebKit Commit Bot 2015-01-19 21:09:06 PST
All reviewed patches have been landed.  Closing bug.