Bug 140658

Summary: [iOS] Only use LLVM static library arguments on 64-bit builds of libllvmForJSC.dylib
Product: WebKit Reporter: David Kilzer (:ddkilzer) <ddkilzer>
Component: JavaScriptCoreAssignee: David Kilzer (:ddkilzer) <ddkilzer>
Status: RESOLVED FIXED    
Severity: Normal CC: ap, commit-queue, dbates, fpizlo, ggaren, mrowe
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch v1 none

David Kilzer (:ddkilzer)
Reported 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.
Attachments
Patch v1 (1.91 KB, patch)
2015-01-19 20:08 PST, David Kilzer (:ddkilzer) 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
David Kilzer (:ddkilzer)
Comment 1 2015-01-19 20:08:26 PST
Created attachment 244958 [details] Patch v1
WebKit Commit Bot
Comment 2 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>
WebKit Commit Bot
Comment 3 2015-01-19 21:09:06 PST
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.