The zorg change looks good.<br><br><div class="gmail_quote">On Mon Dec 22 2014 at 1:03:30 PM Jay Foad <<a href="mailto:jay.foad@gmail.com">jay.foad@gmail.com</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">How about tweaking the compiler-rt cmakefiles so that if lsan is not<br>
supported, the target check-lsan still exists but does nothing? I've<br>
attached a patch that does this. (I don't know much about cmake so<br>
there might be a better way of doing it.)<br>
<br>
Alternatively, can I change the zorg build script so that "run<br>
sanitizer tests in gcc build" doesn't try to run check-lsan etc unless<br>
we're on x86-64? I've attached a patch for this too.<br>
<br>
I can submit either of these patches via phabricator if you'd prefer.<br>
<br>
Thanks,<br>
Jay.<br>
<br>
On 1 December 2014 at 12:58, Evgeniy Stepanov <<a href="mailto:eugenis@google.com" target="_blank">eugenis@google.com</a>> wrote:<br>
> I think check-all-compiler-rt target would be useful on its own, but on a<br>
> buildbot it would result in a single "step" instead of multiple smaller<br>
> "steps", and even less informative messages. I don't really mind, "run<br>
> sanitizer tests in gcc build" is a single step anyway. I would not want that<br>
> to happen to the following "check-asan", "check-msan", etc steps though.<br>
><br>
> With ninja, we could filter against "ninja -t targets".<br>
><br>
><br>
><br>
> On Mon Dec 01 2014 at 3:15:27 PM Jay Foad <<a href="mailto:jay.foad@gmail.com" target="_blank">jay.foad@gmail.com</a>> wrote:<br>
>><br>
>> Hi,<br>
>><br>
>> Currently the first stage ("run sanitizer tests in gcc build") of the<br>
>> sanitizer-ppc64-linux1 buildbot is only failing because of:<br>
>><br>
>> + cd clang_build<br>
>> + make -j16 check-lsan<br>
>> make: *** No rule to make target `check-lsan'.  Stop.<br>
>> + echo @@@STEP_FAILURE@@@<br>
>> @@@STEP_FAILURE@@@<br>
>> + cd clang_build<br>
>> + make -j16 check-msan<br>
>> make: *** No rule to make target `check-msan'.  Stop.<br>
>> + echo @@@STEP_FAILURE@@@<br>
>> @@@STEP_FAILURE@@@<br>
>> + cd clang_build<br>
>> + make -j16 check-tsan<br>
>> make: *** No rule to make target `check-tsan'.  Stop.<br>
>> + echo @@@STEP_FAILURE@@@<br>
>> @@@STEP_FAILURE@@@<br>
>> + cd clang_build<br>
>> + make -j16 check-ubsan<br>
>> make: *** No rule to make target `check-ubsan'.  Stop.<br>
>> + echo @@@STEP_WARNINGS@@@<br>
>> @@@STEP_WARNINGS@@@<br>
>> + cd clang_build<br>
>> + make -j16 check-dfsan<br>
>> make: *** No rule to make target `check-dfsan'.  Stop.<br>
>> + echo @@@STEP_WARNINGS@@@<br>
>> @@@STEP_WARNINGS@@@<br>
>><br>
>> The problem is that the buildbot script<br>
>> (zorg/buildbot/builders/<u></u>sanitizers/buildbot_cmake.sh) explicitly lists<br>
>> all the sanitizer check-* targets, but only some of them are supported<br>
>> on non-x86 configurations. It seems silly for the buildbot script to<br>
>> duplicate all the logic from compiler-rt's cmake/config-ix.cmake about<br>
>> which sanitizers are supported on which targets.<br>
>><br>
>> Maybe compiler-rt could have a new check-all-compiler-rt target, which<br>
>> runs the checks for all supported sanitizers? Is it possible to<br>
>> implement this (it would be a lit target that subsumes a bunch of<br>
>> other lit targets)?<br>
>><br>
>> Any other ideas?<br>
>><br>
>> Thanks,<br>
>> Jay.<br>
</blockquote></div>