[zorg] r195058 - Based on the patch proposed by Rick Foos!

Galina Kistanova gkistanova at gmail.com
Wed Nov 27 10:15:33 PST 2013


It looks like setting up C_COMPILER, CXX_COMPILER, and such locally on a
buildslave is the right thing to do.
The proper change would be remove these options from the build factory.

Rick, could you set up your slave accordingly, please?

For now, just to keep things running and to avoid extra pressure on Rick, I
temporarily patch the build factory to setup DCMAKE_C_COMPILER options only
if we are building
with sanitizers, and will remove this change once Rick will confirm his
buildslave configured.

If this fine with everyone, I'll apply the workaround tonight when I'll be
restarting the buildmaster.
If someone sees issues with this approach, please speak up.

Thanks

Galina



On Fri, Nov 22, 2013 at 10:10 AM, Dmitri Gribenko <gribozavr at gmail.com>wrote:

> On Fri, Nov 22, 2013 at 7:11 AM, Rick Foos <rfoos at codeaurora.org> wrote:
> > Maybe a cmakeOptions like options with the default like it used to be
> > (excluding C and CXX compiler), and allow the user to add options?
>
> Maybe.  Or maybe add DCMAKE_C_COMPILER options only if we are building
> with sanitizers?  Whichever is convenient, seems like both will work
> for me.
>
> 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>*/
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20131127/226bb7ea/attachment.html>


More information about the llvm-commits mailing list