Add new library dependency for LLVMForJavaScriptCore dylib
Created attachment 266306 [details] Patch
Comment on attachment 266306 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=266306&action=review r=me > Source/JavaScriptCore/ChangeLog:7 > + Adding this dependency should be backwards compatible, since LLVM has build and typo: build ==> built.
Have you tested that this change builds successfully with new and old LLVM?
Yes, I tried this with older LLVMs for iOS and OS X. The buildbots seems to be happy too and they are still using an older LLVM version.
Comment on attachment 266306 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=266306&action=review > Source/JavaScriptCore/Configurations/LLVMForJSC.xcconfig:35 > -LLVM_LIBS_macosx = -lLLVMLinker -lLLVMipo -lLLVMVectorize -lLLVMIRReader -lLLVMBitReader -lLLVMAsmParser -lLLVMSelectionDAG -lLLVMAsmPrinter -lLLVMMCParser -lLLVMDebugInfo -lLLVMOption -lLLVMCodeGen -lLLVMScalarOpts -lLLVMInstCombine -lLLVMTransformUtils -lLLVMipa -lLLVMAnalysis -lLLVMMCDisassembler -lLLVMMCJIT -lLLVMTarget -lLLVMRuntimeDyld -lLLVMExecutionEngine -lLLVMMC -lLLVMObject -lLLVMCore -lLLVMSupport -lLLVMX86Disassembler -lLLVMX86CodeGen -lLLVMX86AsmParser -lLLVMX86Desc -lLLVMX86Info -lLLVMX86AsmPrinter -lLLVMX86Utils; > +LLVM_LIBS_macosx = -lLLVMInstrumentation -lLLVMLinker -lLLVMipo -lLLVMVectorize -lLLVMIRReader -lLLVMBitReader -lLLVMAsmParser -lLLVMSelectionDAG -lLLVMAsmPrinter -lLLVMMCParser -lLLVMDebugInfo -lLLVMOption -lLLVMCodeGen -lLLVMScalarOpts -lLLVMInstCombine -lLLVMTransformUtils -lLLVMipa -lLLVMAnalysis -lLLVMMCDisassembler -lLLVMMCJIT -lLLVMTarget -lLLVMRuntimeDyld -lLLVMExecutionEngine -lLLVMMC -lLLVMObject -lLLVMCore -lLLVMSupport -lLLVMX86Disassembler -lLLVMX86CodeGen -lLLVMX86AsmParser -lLLVMX86Desc -lLLVMX86Info -lLLVMX86AsmPrinter -lLLVMX86Utils; Is this change necessary?
Yes, this is a generic/target independent new dependency on this library.
Created attachment 266370 [details] Patch
Fixed typo.
Created attachment 266374 [details] Patch
Limit fix to iOS.
Comment on attachment 266374 [details] Patch Clearing flags on attachment: 266374 Committed r192918: <http://trac.webkit.org/changeset/192918>
All reviewed patches have been landed. Closing bug.