<div dir="ltr">This essentially disabled check-scudo on the linux bots: <div>Before: <a href="http://lab.llvm.org:8011/builders/sanitizer-x86_64-linux/builds/24296">http://lab.llvm.org:8011/builders/sanitizer-x86_64-linux/builds/24296</a> (has check-scudo and 42 items in total)</div><div>After: <a href="http://lab.llvm.org:8011/builders/sanitizer-x86_64-linux/builds/24297">http://lab.llvm.org:8011/builders/sanitizer-x86_64-linux/builds/24297</a> (no check-scudo and 41 items in total)</div><div><br></div><div>I frankly don't understand why this works for <span style="font-size:12.8px">SKIP_MSAN/</span><span style="font-size:12.8px">SKIP_LSAN/</span><span style="font-size:12.8px">SKIP_DFSAN given that they are all defined as being 1 by default</span></div><div><div><span style="font-size:12.8px">SKIP_MSAN=${SKIP_MSAN:-1}</span></div><div><span style="font-size:12.8px">SKIP_LSAN=${SKIP_LSAN:-1}</span></div><div><span style="font-size:12.8px">SKIP_DFSAN=${SKIP_DFSAN:-1}</span></div><div><span style="font-size:12.8px">SKIP_SCUDO=${SKIP_SCUDO:-1}</span></div><div><span style="font-size:12.8px">and then only the ppc bots define them (again) to be 1... </span></div><div><span style="font-size:12.8px"><br></span></div><div><span style="font-size:12.8px">Thoughts? </span></div><div style="font-size:12.8px"><br></div></div><div><span style="font-size:12.8px"><br></span></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Jun 8, 2016 at 10:40 AM, Bill Seurer via llvm-commits <span dir="ltr"><<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Author: seurer<br>
Date: Wed Jun  8 12:40:20 2016<br>
New Revision: 272178<br>
<br>
URL: <a href="http://llvm.org/viewvc/llvm-project?rev=272178&view=rev" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project?rev=272178&view=rev</a><br>
Log:<br>
[powerpc] Add ability to bypass scudo tests<br>
<br>
The check-scudo target is not available on all targets.<br>
<br>
<br>
Modified:<br>
    zorg/trunk/zorg/buildbot/builders/sanitizers/buildbot_cmake.sh<br>
    zorg/trunk/zorg/buildbot/builders/sanitizers/buildbot_selector.py<br>
<br>
Modified: zorg/trunk/zorg/buildbot/builders/sanitizers/buildbot_cmake.sh<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/zorg/trunk/zorg/buildbot/builders/sanitizers/buildbot_cmake.sh?rev=272178&r1=272177&r2=272178&view=diff" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project/zorg/trunk/zorg/buildbot/builders/sanitizers/buildbot_cmake.sh?rev=272178&r1=272177&r2=272178&view=diff</a><br>
==============================================================================<br>
--- zorg/trunk/zorg/buildbot/builders/sanitizers/buildbot_cmake.sh (original)<br>
+++ zorg/trunk/zorg/buildbot/builders/sanitizers/buildbot_cmake.sh Wed Jun  8 12:40:20 2016<br>
@@ -178,6 +178,7 @@ HAVE_NINJA=${HAVE_NINJA:-1}<br>
 SKIP_MSAN=${SKIP_MSAN:-1}<br>
 SKIP_LSAN=${SKIP_LSAN:-1}<br>
 SKIP_DFSAN=${SKIP_DFSAN:-1}<br>
+SKIP_SCUDO=${SKIP_SCUDO:-1}<br>
 if [ "$PLATFORM" == "Linux" -a $HAVE_NINJA == 1 ]; then<br>
   echo @@@BUILD_STEP build with ninja@@@<br>
   if [ ! -d llvm_build_ninja ]; then<br>
@@ -196,8 +197,10 @@ if [ "$PLATFORM" == "Linux" -a $HAVE_NIN<br>
   (cd llvm_build_ninja && ninja check-tsan) || echo @@@STEP_FAILURE@@@<br>
   echo @@@BUILD_STEP ninja check-ubsan@@@<br>
   (cd llvm_build_ninja && ninja check-ubsan) || echo @@@STEP_FAILURE@@@<br>
-  echo @@@BUILD_STEP ninja check-scudo@@@<br>
-  (cd llvm_build_ninja && ninja check-scudo) || echo @@@STEP_FAILURE@@@<br>
+  if [ "$SKIP_SCUDO" != 1 ]; then<br>
+    echo @@@BUILD_STEP ninja check-scudo@@@<br>
+    (cd llvm_build_ninja && ninja check-scudo) || echo @@@STEP_FAILURE@@@<br>
+  fi<br>
   if [ "$SKIP_MSAN" != 1 ]; then<br>
     echo @@@BUILD_STEP ninja check-msan@@@<br>
     (cd llvm_build_ninja && ninja check-msan) || echo @@@STEP_FAILURE@@@<br>
<br>
Modified: zorg/trunk/zorg/buildbot/builders/sanitizers/buildbot_selector.py<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/zorg/trunk/zorg/buildbot/builders/sanitizers/buildbot_selector.py?rev=272178&r1=272177&r2=272178&view=diff" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project/zorg/trunk/zorg/buildbot/builders/sanitizers/buildbot_selector.py?rev=272178&r1=272177&r2=272178&view=diff</a><br>
==============================================================================<br>
--- zorg/trunk/zorg/buildbot/builders/sanitizers/buildbot_selector.py (original)<br>
+++ zorg/trunk/zorg/buildbot/builders/sanitizers/buildbot_selector.py Wed Jun  8 12:40:20 2016<br>
@@ -24,8 +24,8 @@ BOT_ASSIGNMENT = {<br>
 }<br>
<br>
 BOT_ADDITIONAL_ENV = {<br>
-    'sanitizer-ppc64le-linux': { 'HAVE_NINJA': '1', 'CHECK_LIBCXX': '0', 'CHECK_LLD': '0', 'SKIP_MSAN': '1', 'SKIP_LSAN': '1', 'SKIP_DFSAN': '1' },<br>
-    'sanitizer-ppc64be-linux': { 'HAVE_NINJA': '1', 'CHECK_LIBCXX': '0', 'CHECK_LLD': '0' },<br>
+    'sanitizer-ppc64le-linux': { 'HAVE_NINJA': '1', 'CHECK_LIBCXX': '0', 'CHECK_LLD': '0', 'SKIP_MSAN': '1', 'SKIP_LSAN': '1', 'SKIP_DFSAN': '1', 'SKIP_SCUDO': '1' },<br>
+    'sanitizer-ppc64be-linux': { 'HAVE_NINJA': '1', 'CHECK_LIBCXX': '0', 'CHECK_LLD': '0', 'SKIP_SCUDO': '1' },<br>
     'sanitizer-x86_64-linux': { 'HAVE_NINJA' : '1', 'BUILD_ANDROID' : '1', 'RUN_ANDROID' : '1' },<br>
     'sanitizer-x86_64-linux-bootstrap': {},<br>
     'sanitizer-x86_64-linux-fast': {},<br>
<br>
<br>
_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@lists.llvm.org">llvm-commits@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits</a><br>
</blockquote></div><br></div>