<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:0 0 0 .8ex;border-left:1px #ccc 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">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">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>