<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Fri, Dec 16, 2016 at 12:45 AM, Akira Hatanaka <span dir="ltr"><<a href="mailto:ahatanaka@apple.com" target="_blank">ahatanaka@apple.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="word-wrap:break-word"><div dir="auto" style="word-wrap:break-word"><div>It looks like it failed again. I’m still looking.</div></div></div></blockquote><div><br></div><div>Well, the bot needs to run once with my change in. In that run, the .ll file is still around and the tests will still fail, but the .cpp test will remove it. On the first build _after_ my fix, it should cycle green.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="word-wrap:break-word"><div dir="auto" style="word-wrap:break-word"><div><div class="h5"><div><br></div><div><blockquote type="cite"><div>On Dec 15, 2016, at 9:14 PM, Nico Weber <<a href="mailto:thakis@chromium.org" target="_blank">thakis@chromium.org</a>> wrote:</div><br class="m_-6059663614410222331Apple-interchange-newline"><div><div dir="ltr">I gave this a shot in 289924. Please check that's the right thing.</div><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Dec 16, 2016 at 12:09 AM, Nico Weber <span dir="ltr"><<a href="mailto:thakis@chromium.org" target="_blank">thakis@chromium.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><span>On Fri, Dec 16, 2016 at 12:08 AM, Nico Weber <span dir="ltr"><<a href="mailto:thakis@chromium.org" target="_blank">thakis@chromium.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">I think the problem was that you didn't pipe the output to /dev/null or -, and so a .ll file got created which now confuses the bots.<div><br></div><div>I suggest:</div><div>1. Add `// RUN: rm -f right_path.ll` at the top for a while (look through magic lit vars to find one that expands to the right thing)</div></div></blockquote><div><br></div></span><div>I think you want `rm -f %S/default-expr-arguments-3.ll<wbr>`</div><div><div class="m_-6059663614410222331h5"><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div>2. If the intent for this is to be a -verify check, add that back</div><div>3. If not, remove expected-no-diagnostics</div><div>4. If not, is this a FileCheck test? If so, pipe output to FileCheck, add CHECK lines</div><div>5. If it isn't FileCheck either, what does this test?</div></div><div class="m_-6059663614410222331m_411664322990746105HOEnZb"><div class="m_-6059663614410222331m_411664322990746105h5"><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Dec 16, 2016 at 12:05 AM, Nico Weber <span dir="ltr"><<a href="mailto:thakis@chromium.org" target="_blank">thakis@chromium.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Are you aware that this is still failing on all the bots? (<a href="http://lab.llvm.org:8011/builders/clang-atom-d525-fedora-rel/builds/493" target="_blank">http://lab.llvm.org:8011/buil<wbr>ders/clang-atom-d525-fedora-re<wbr>l/builds/493</a> eg)</div><div class="m_-6059663614410222331m_411664322990746105m_6090275564076196933HOEnZb"><div class="m_-6059663614410222331m_411664322990746105m_6090275564076196933h5"><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Dec 15, 2016 at 11:18 PM, Akira Hatanaka via cfe-commits <span dir="ltr"><<a href="mailto:cfe-commits@lists.llvm.org" target="_blank">cfe-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: ahatanak<br>
Date: Thu Dec 15 22:18:53 2016<br>
New Revision: 289919<br>
<br>
URL: <a href="http://llvm.org/viewvc/llvm-project?rev=289919&view=rev" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-pr<wbr>oject?rev=289919&view=rev</a><br>
Log:<br>
Remove "-disable-llvm-optzns -verify" from the RUN line.<br>
<br>
Modified:<br>
    cfe/trunk/test/SemaTemplate/de<wbr>fault-expr-arguments-3.cpp<br>
<br>
Modified: cfe/trunk/test/SemaTemplate/de<wbr>fault-expr-arguments-3.cpp<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/cfe/trunk/test/SemaTemplate/default-expr-arguments-3.cpp?rev=289919&r1=289918&r2=289919&view=diff" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-pr<wbr>oject/cfe/trunk/test/SemaTempl<wbr>ate/default-expr-arguments-3.c<wbr>pp?rev=289919&r1=289918&r2=289<wbr>919&view=diff</a><br>
==============================<wbr>==============================<wbr>==================<br>
--- cfe/trunk/test/SemaTemplate/de<wbr>fault-expr-arguments-3.cpp (original)<br>
+++ cfe/trunk/test/SemaTemplate/de<wbr>fault-expr-arguments-3.cpp Thu Dec 15 22:18:53 2016<br>
@@ -1,4 +1,4 @@<br>
-// RUN: %clang_cc1 -std=c++14 -emit-llvm -disable-llvm-optzns -verify %s<br>
+// RUN: %clang_cc1 -std=c++14 -emit-llvm -o - %s<br>
 // expected-no-diagnostics<br>
<br>
 namespace PR28795 {<br>
<br>
<br>
______________________________<wbr>_________________<br>
cfe-commits mailing list<br>
<a href="mailto:cfe-commits@lists.llvm.org" target="_blank">cfe-commits@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/<wbr>mailman/listinfo/cfe-commits</a><br>
</blockquote></div><br></div>
</div></div></blockquote></div><br></div>
</div></div></blockquote></div></div></div><br></div></div>
</blockquote></div><br></div>
</div></blockquote></div><br></div></div></div></div></blockquote></div><br></div></div>