[llvm-commits] [zorg] r172885 - /zorg/trunk/zorg/buildbot/builders/ClangBuilder.py

[NAKAMURA Takumi]

clang-x86_64-linux-vg became green.
http://lab.llvm.org:8011/builders/clang-x86_64-linux-vg/builds/648

David, could you reconstruct test steps to exclude llvm out of valgrind?

...Takumi

2013/1/19 Galina Kistanova <gkistanova at gmail.com>:
> Done.
>
> Thanks
>
> Galina
>
> On Fri, Jan 18, 2013 at 7:47 PM, NAKAMURA Takumi <geek4civic at gmail.com>
> wrote:
>>
>> Galina, could you reconfigure the llvmlab for this?
>> Then, I will begin watching;
>> http://lab.llvm.org:8011/builders/clang-x86_64-linux-vg
>>
>> ...Takumi
>>
>> 2013/1/19 NAKAMURA Takumi <geek4civic at gmail.com>:
>> > Author: chapuni
>> > Date: Fri Jan 18 21:39:07 2013
>> > New Revision: 172885
>> >
>> > URL: http://llvm.org/viewvc/llvm-project?rev=172885&view=rev
>> > Log:
>> > ClangBuilder: Use LIT_ARGS=clangTestArgs, or valgrind would not be
>> > enabled.
>> >
>> > FIXME: Valgrind affects also llvm tests. Is it expected?
>> >
>> > Modified:
>> >     zorg/trunk/zorg/buildbot/builders/ClangBuilder.py
>> >
>> > Modified: zorg/trunk/zorg/buildbot/builders/ClangBuilder.py
>> > URL:
>> > http://llvm.org/viewvc/llvm-project/zorg/trunk/zorg/buildbot/builders/ClangBuilder.py?rev=172885&r1=172884&r2=172885&view=diff
>> >
>> > ==============================================================================
>> > --- zorg/trunk/zorg/buildbot/builders/ClangBuilder.py (original)
>> > +++ zorg/trunk/zorg/buildbot/builders/ClangBuilder.py Fri Jan 18
>> > 21:39:07 2013
>> > @@ -208,7 +208,7 @@
>> >                                                workdir=llvm_1_objdir,
>> >                                                env=merged_env))
>> >
>> > -    clangTestArgs = llvmTestArgs = '-v -j %s' % jobs
>> > +    clangTestArgs = '-v -j %s' % jobs
>> >      if valgrind:
>> >          clangTestArgs += ' --vg'
>> >          if valgrindLeakCheck:
>> > @@ -220,8 +220,7 @@
>> >      if test:
>> >          f.addStep(LitTestCommand(name='check-all',
>> >                                     command=[make, "check-all",
>> > "VERBOSE=1",
>> > -
>> > WithProperties("LIT_ARGS=%s" % llvmTestArgs),
>> > -
>> > WithProperties("TESTARGS=%s" % clangTestArgs),
>> > +
>> > WithProperties("LIT_ARGS=%s" % clangTestArgs),
>> >
>> > WithProperties("EXTRA_TESTDIRS=%s" % extraTestDirs)],
>> >                                     description=["checking"],
>> >                                     descriptionDone=["checked"],
>> > @@ -323,8 +322,7 @@
>> >      if test:
>> >          f.addStep(LitTestCommand(name='check-all',
>> >                                     command=[make, "check-all",
>> > "VERBOSE=1",
>> > -
>> > WithProperties("LIT_ARGS=%s" % llvmTestArgs),
>> > -
>> > WithProperties("TESTARGS=%s" % clangTestArgs),
>> > +
>> > WithProperties("LIT_ARGS=%s" % clangTestArgs),
>> >
>> > WithProperties("EXTRA_TESTDIRS=%s" % extraTestDirs)],
>> >                                     description=["checking"],
>> >                                     descriptionDone=["checked"],
>> >
>> >
>> > _______________________________________________
>> > llvm-commits mailing list
>> > llvm-commits at cs.uiuc.edu
>> > http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits
>
>