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:

JavaScriptCore

Assignee:

David Kilzer (:ddkilzer) <ddkilzer>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

ap, commit-queue, dbates, fpizlo, ggaren, mrowe

Priority:

Version:

528+ (Nightly build)

Hardware:

Unspecified

OS:

Unspecified

Attachments:

Description	Flags
Patch v1	none

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.