[PATCH] Zorg: getLLDBuildFactory() now takes an optional 'env' parameter

Mikael Lyngvig mikael at lyngvig.org
Sat Dec 7 02:48:42 PST 2013


I am finally getting somewhere on my project of setting up a PowerPC lld
builder and for that I need this patch to be included in Zorg.  It simply
adds the 'env' parameter to getLLDBuildFactory() that most of the other
build factory functions have.

The change is intended to be non-intrusive in the sense that everything
should work as before.  One exception may be that I have copied over the
TERM=dumb code from getLLVMBuildFactory() (which makes Clang not use colors
in its output).  If this is undesirable, please let me know and I'll fix
it.  It might break the lld test suite or something, I don't know enough
about lld to say.

P.S. I plan to build using GCC v4.8.2, not Clang.  I think it is a good
thing to have a GNU builder seeing that there are already three Clang
builders of LLD.


-- Mikael
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20131207/e006625c/attachment.html>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: zorg-lldbuilder-env.patch
Type: application/octet-stream
Size: 2486 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20131207/e006625c/attachment.obj>


More information about the llvm-commits mailing list