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

Dmitri Gribenko gribozavr at gmail.com
Mon Dec 9 11:21:53 PST 2013


On Sat, Dec 7, 2013 at 2:48 AM, Mikael Lyngvig <mikael at lyngvig.org> wrote:
> 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.

+                   'C'    : "clang",

This should be 'CC'.

Also, I think that this change might disable ccache on my builders,
because this change overrides system's CC and CXX when the environment
configured in zorg is empty.  I think that there should be no defaults
for CC and CXX.  These can be overridden in per-builder configuration.

Dmitri

-- 
main(i,j){for(i=2;;i++){for(j=2;j<i;j++){if(!(i%j)){j=0;break;}}if
(j){printf("%d\n",i);}}} /*Dmitri Gribenko <gribozavr at gmail.com>*/



More information about the llvm-commits mailing list