<div dir="ltr">Woops, I put it back in r215044.</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Wed, Aug 6, 2014 at 3:26 PM, Tobias Grosser <span dir="ltr"><<a href="mailto:tobias@grosser.es" target="_blank">tobias@grosser.es</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="">On 05/08/2014 22:49, Reid Kleckner wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Author: rnk<br>
Date: Tue Aug  5 15:49:12 2014<br>
New Revision: 214924<br>
<br>
URL: <a href="http://llvm.org/viewvc/llvm-project?rev=214924&view=rev" target="_blank">http://llvm.org/viewvc/llvm-<u></u>project?rev=214924&view=rev</a><br>
Log:<br>
Make crash diagnostics on Windows the tiniest bit more useful<br>
<br>
This escapes any backslashes in the executable path and fixes an issue<br>
with a trailing quote when the main file name had to be quoted during<br>
printing.<br>
<br>
It's impossible to test this without putting backslashes or quotes into<br>
the executable path, so I didn't add automated tests.<br>
<br>
The crash diagnostics are still only useful if you're using bash on<br>
Windows, though.  This should probably be writing a batch file instead.<br>
</blockquote>
<br></div>
This also broke the lnt nightly testers.<br>
<br>
I now get:<br>
<br>
mysandbox/bin/lnt  runtest nt --sandbox nt --cc build/bin/clang --cxx build/bin/clang++ --test-suite test-suite/ --only-test=MultiSource/<u></u>Benchmarks/MallocBench/gs<br>
<br>
Traceback (most recent call last):<br>
  File "mysandbox/bin/lnt", line 9, in <module><br>
    load_entry_point('LNT==0.4.<u></u>1dev', 'console_scripts', 'lnt')()<br>
  File "/home/grosser/Projects/polly/<u></u>lnt/lnt/lnttool/main.py", line 443, in main<br>
    return tool.main(*args, **kwargs)<br>
  File "/home/grosser/Projects/polly/<u></u>lnt/lnt/util/multitool.py", line 60, in main<br>
    self.commands[cmd]('%s %s' % (os.path.basename(args[0]), cmd), args[2:])<br>
  File "/home/grosser/Projects/polly/<u></u>lnt/lnt/lnttool/main.py", line 155, in action_runtest<br>
    test_instance.run_test('%s %s' % (name, test_name), args)<br>
  File "/home/grosser/Projects/polly/<u></u>lnt/lnt/tests/nt.py", line 1491, in run_test<br>
    report = run_test(nick, None, config)<br>
  File "/home/grosser/Projects/polly/<u></u>lnt/lnt/tests/nt.py", line 847, in run_test<br>
    make_variables, public_make_variables = config.compute_run_make_<u></u>variables()<br>
  File "/home/grosser/Projects/polly/<u></u>lnt/lnt/tests/nt.py", line 188, in compute_run_make_variables<br>
    cc_info = self.cc_info<br>
  File "/home/grosser/Projects/polly/<u></u>lnt/lnt/tests/nt.py", line 145, in cc_info<br>
    self.target_flags)<br>
  File "/home/grosser/Projects/polly/<u></u>lnt/lnt/testing/util/<u></u>compilers.py", line 74, in get_cc_info<br>
    cc1_binary, = m.groups()<br>
AttributeError: 'NoneType' object has no attribute 'groups'<br>
<br>
It seems the reason is that a white space at the beginning of the last line of 'clang -v -E -x c /dev/null -###' is not longer emitted, but LNT<br>
seems to match for exactly this white space. I am not sure if and why<br>
this white space was there and if it carried some meaning, but in case this change on the clang side was intentional, I attached a patch to fix LNT.<br>
<br>
Cheers,<br>
Tobias<br>
<br>_______________________________________________<br>
cfe-commits mailing list<br>
<a href="mailto:cfe-commits@cs.uiuc.edu">cfe-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits</a><br>
<br></blockquote></div><br></div>