<div dir="ltr">On Wed, Jun 25, 2014 at 8:30 AM, Chris Matthews <span dir="ltr"><<a href="mailto:chris4000@gmail.com" target="_blank">chris4000@gmail.com</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote">
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div style="word-wrap:break-word"><div>I think the message failed to parse output is misleading.  </div>
<div><br></div><div>The problem that I have experienced several times is that no output is printed, so all I get is a "compile.py:71: error: failed to parse output: “ and nothing else, which then makes it look like LNT had a problem, not the subprocess.</div>
<div><br></div><div>For instance, this is a snippet from a recent log:</div><div><br></div><div><div class=""><div>compile.py:71: error: failed to parse output: </div></div><div>compile.py:74: error: stderr isn't empty: runN: ERROR: child returned error: 65.</div>
<div class=""><div>compile.py:78: error: res != 0: 1</div></div></div><div><br></div><div><br></div><div>Not super helpful.</div></div></blockquote><div><br></div><div>That makes sense, and definitely not super helpful.</div>
<div><br></div><div>I put in a slightly different patch to make this more readable here:</div><div>  <a href="http://llvm.org/viewvc/llvm-project?view=revision&revision=211719">http://llvm.org/viewvc/llvm-project?view=revision&revision=211719</a></div>
<div><br></div><div>Steven, can you verify that resolves the problem you were seeing?</div><div><br></div><div>Now you get output like:</div><div>----</div><div><div>2014-06-25 10:15:27: ERROR: command failed with stderr:</div>
<div>--</div><div>/Users/ddunbar/test-suite-externals/lnt-compile-suite-src/intel/single-file/403.gcc/combine.c:12778:2: error: foo</div><div>#error foo</div><div> ^</div><div>1 error generated.</div><div>runN: ERROR: child returned error: 1.</div>
<div>--</div><div>----<br></div></div><div>or</div><div>----</div><div>2014-06-25 10:15:50: ERROR: command had unexpected output on stderr:</div><div>--</div><div>/Users/ddunbar/test-suite-externals/lnt-compile-suite-src/intel/single-file/403.gcc/combine.c:12778:2: warning: foo [-W#warnings]</div>
<div>#warning foo</div><div> ^</div><div>1 warning generated.</div><div>--</div><div>----</div><div><br></div><div> - Daniel</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
<div style="word-wrap:break-word"><div><div class="h5"><br><div><blockquote type="cite"><div>On Jun 25, 2014, at 7:52 AM, Steven Wu <<a href="mailto:stevenwu@apple.com" target="_blank">stevenwu@apple.com</a>> wrote:</div>
<br><div><div style="word-wrap:break-word">I agree that the original code works but it is doing more work than necessary and throw ugly python errors. This is a performance & cosmetic fix.<div><br></div><div>Steven</div>
<div><br><div><blockquote type="cite"><div>On Jun 24, 2014, at 10:27 AM, Daniel Dunbar <<a href="mailto:daniel@zuster.org" target="_blank">daniel@zuster.org</a>> wrote:</div><br><div><div dir="ltr">It isn't clear to me this is the right patch. Where are you seeing "python errors" being dumped during "runtest compile"?<div>
<br></div><div>If I inject an error into one of the source files, what I see is this:</div>
<div>--</div><div><div>$ lnt runtest compile --sandbox $(timestamp) --cc $(which clang) --test-externals ~/test-suite-externals --test-subdir lnt-compile-suite-src/intel --test-filter=compile/403.gcc/combine.c/irgen --flags-to-test="-O0"</div>

<div>compile.py:779: note: inferred C++ compiler under test as: '/usr/bin/clang++'</div><div>2014-06-24 17:25:04: creating sandbox: '2014-06-24_10-25'</div><div>compile.py:893: note: inferred run order to be: '600.0.34'</div>

<div>2014-06-24 10:25:04: INFO: Loading config file: /Users/ddunbar/test-suite-externals/lnt-compile-suite-src/intel/project_list.json</div><div>2014-06-24 10:25:04: INFO: run started</div><div>2014-06-24 10:25:04: INFO: using CC: '/usr/bin/clang'</div>

<div>2014-06-24 10:25:04: INFO: using CXX: '/usr/bin/clang++'</div><div>compile.py:71: error: failed to parse output: </div><div><br></div><div>compile.py:74: error: stderr isn't empty: /Users/ddunbar/test-suite-externals/lnt-compile-suite-src/intel/single-file/403.gcc/combine.c:12776:1: error: unknown type name 'xxxx'</div>

<div>xxxx</div><div>^</div><div>/Users/ddunbar/test-suite-externals/lnt-compile-suite-src/intel/single-file/403.gcc/combine.c:12777:5: error: expected ';' after top level declarator</div><div>xxxx</div><div>    ^</div>

<div>    ;</div><div>2 errors generated.</div><div>runN: ERROR: child returned error: 1.</div><div><br></div><div><br></div><div>compile.py:78: error: res != 0: 1</div><div>--</div></div><div>which isn't pretty, but it works.</div>

<div><br></div><div> - Daniel</div><div><br></div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Tue, Jun 24, 2014 at 9:34 AM, Steven Wu <span dir="ltr"><<a href="mailto:stevenwu@apple.com" target="_blank">stevenwu@apple.com</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">Sorry I forgot to attach the patch in the previous email. Now attached.<br>

<br><br>
> On Jun 23, 2014, at 6:06 PM, Steven Wu <<a href="mailto:stevenwu@apple.com" target="_blank">stevenwu@apple.com</a>> wrote:<br>
><br>
> LNT currently throws python errors when runN failed in compile.py. It throws error messages about it cannot parse stdout, stderr is not empty and runN does not return 0.<br>
> Ideally, the check after runN should performed in the opposite order. stdout shouldn’t be parsed if runN does not return 0. Since it is expected that runN can fail during the test, g_log.error should be used to print the error instead of throw python error messages.<br>


><br>
> Please commit for me after review.<br>
<br>
<br>_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@cs.uiuc.edu" target="_blank">llvm-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a><br>
<br></blockquote></div><br></div>
</div></blockquote></div><br></div></div>_______________________________________________<br>llvm-commits mailing list<br><a href="mailto:llvm-commits@cs.uiuc.edu" target="_blank">llvm-commits@cs.uiuc.edu</a><br><a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a><br>
</div></blockquote></div><br></div></div></div></blockquote></div><br></div></div>