[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