<div class="gmail_extra"><div class="gmail_quote">On Fri, Aug 31, 2012 at 11:53 AM, Daniel Dunbar <span dir="ltr"><<a href="mailto:daniel@zuster.org" target="_blank" class="cremed">daniel@zuster.org</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">On Fri, Aug 31, 2012 at 11:23 AM, David Blaikie <<a href="mailto:dblaikie@gmail.com" class="cremed">dblaikie@gmail.com</a>> wrote:<br>

> Daniel (& anyone else),<br>
><br>
> Do you know if the documentation here (<br>
> <a href="http://llvm.org/docs/TestingGuide.html#rtcustom" target="_blank" class="cremed">http://llvm.org/docs/TestingGuide.html#rtcustom</a> /<br>
> <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/docs/TestingGuide.html?view=diff&r1=36059&r2=36060" target="_blank" class="cremed">http://llvm.org/viewvc/llvm-project/llvm/trunk/docs/TestingGuide.html?view=diff&r1=36059&r2=36060</a><br>

> ) is still valid? It doesn't appear to be, since my change below<br>
> doesn't do particularly interesting quoting/escaping gymnastics &<br>
> seems to work fine.<br>
><br>
> Any ideas? Should we update the documentation? Does anyone know<br>
> what/when/how/why this changed?<br>
<br>
</div>What is actually going on here is that the LLVM test suite and the<br>
Clang test suite use a different style of test. </blockquote><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
The LLVM test suite is technically written using Tcl syntax, and the<br>
Clang test suite uses shell syntax. The document in question is<br>
referring to the LLVM test suite.<br></blockquote><div><br></div><div>Is this still true? I tried to switch the LLVM test suite to use the ShTest syntax. I thought I had succeeded.</div><div><br></div><div>I've got a TODO to go and delete all the lit code for supporting the TclTest variant.</div>
<div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
The fact that the test suites use different styles is an unfortunate<br>
and poorly documented historical accident. Ideally we would kill off<br>
the Tcl style and only use the shell style.<br>
<br>
Also, what is a "lesser lit-like" test runner?<br>
<span class="HOEnZb"><font color="#888888"><br>
 - Daniel<br>
</font></span><div class="HOEnZb"><div class="h5"><br>
><br>
> Thanks,<br>
> - David<br>
><br>
> On Fri, Aug 31, 2012 at 10:49 AM, David Blaikie <<a href="mailto:dblaikie@gmail.com" class="cremed">dblaikie@gmail.com</a>> wrote:<br>
>> Author: dblaikie<br>
>> Date: Fri Aug 31 12:49:33 2012<br>
>> New Revision: 163009<br>
>><br>
>> URL: <a href="http://llvm.org/viewvc/llvm-project?rev=163009&view=rev" target="_blank" class="cremed">http://llvm.org/viewvc/llvm-project?rev=163009&view=rev</a><br>
>> Log:<br>
>> Fix tests to be more robust (to older versions of grep, lesser lit-like test runners, etc)<br>
>><br>
>> Seems I had a problem with my version of grep, when run by lit, not supporting<br>
>> the \s escape. This seems to fix it for me & I'll be getting the buildbots to<br>
>> run these tests too to keep an eye on them (actually loop-convert tests still<br>
>> fail when run via a make build, so that'll be addressed in a future commit). I<br>
>> could use [[:space:]] to generalize over other whitespace but that seemed<br>
>> unnecessarily verbose when the flexibility wasn't actually required by the<br>
>> current text of the tests.<br>
>><br>
>> Also I just simplified a lot of the loop-convert tests (removing the<br>
>> unecessary temp file deletion at the start, removing the unnecessary && for<br>
>> FileCheck, etc).<br>
>><br>
>> The remove-cstr-calls/basic.cpp changes were necessitated by an out of tree<br>
>> lit-like test runner that's a bit less fantastic about escaping. They were<br>
>> modeled on existing tooling test cases in Clang, with thanks to Manuel Klimek<br>
>> for the pointers.<br>
>><br>
>> Modified:<br>
>>     clang-tools-extra/trunk/test/loop-convert/array.cpp<br>
>>     clang-tools-extra/trunk/test/loop-convert/confidence.cpp<br>
>>     clang-tools-extra/trunk/test/loop-convert/dependency.cpp<br>
>>     clang-tools-extra/trunk/test/loop-convert/iterator.cpp<br>
>>     clang-tools-extra/trunk/test/loop-convert/naming.cpp<br>
>>     clang-tools-extra/trunk/test/loop-convert/negative-iterator.cpp<br>
>>     clang-tools-extra/trunk/test/loop-convert/negative-multi-end-call.cpp<br>
>>     clang-tools-extra/trunk/test/loop-convert/negative-pseudoarray-extra.cpp<br>
>>     clang-tools-extra/trunk/test/loop-convert/negative-pseudoarray.cpp<br>
>>     clang-tools-extra/trunk/test/loop-convert/negative.cpp<br>
>>     clang-tools-extra/trunk/test/loop-convert/nesting.cpp<br>
>>     clang-tools-extra/trunk/test/loop-convert/nocompile.cpp<br>
>>     clang-tools-extra/trunk/test/loop-convert/pseudoarray.cpp<br>
>>     clang-tools-extra/trunk/test/loop-convert/single-iterator.cpp<br>
>>     clang-tools-extra/trunk/test/remove-cstr-calls/basic.cpp<br>
>><br>
>> Modified: clang-tools-extra/trunk/test/loop-convert/array.cpp<br>
>> URL: <a href="http://llvm.org/viewvc/llvm-project/clang-tools-extra/trunk/test/loop-convert/array.cpp?rev=163009&r1=163008&r2=163009&view=diff" target="_blank" class="cremed">http://llvm.org/viewvc/llvm-project/clang-tools-extra/trunk/test/loop-convert/array.cpp?rev=163009&r1=163008&r2=163009&view=diff</a><br>

>> ==============================================================================<br>
>> --- clang-tools-extra/trunk/test/loop-convert/array.cpp (original)<br>
>> +++ clang-tools-extra/trunk/test/loop-convert/array.cpp Fri Aug 31 12:49:33 2012<br>
>> @@ -1,12 +1,11 @@<br>
>> -// RUN: rm -rf %t.cpp<br>
>> -// RUN: grep -Ev "//\s*[A-Z-]+:" %s > %t.cpp<br>
>> -// RUN: loop-convert . %t.cpp -- -I %S/Inputs \<br>
>> -// RUN:         && FileCheck -input-file=%t.cpp %s<br>
>> -// RUN: grep -Ev "//\s*[A-Z-]+:" %s > %t.cpp<br>
>> +// RUN: grep -Ev "// *[A-Z-]+:" %s > %t.cpp<br>
>>  // RUN: cp %t.cpp %t.base<br>
>> -// RUN: loop-convert -count-only . %t.cpp -- -I %S/Inputs > %T/out \<br>
>> -// RUN:         && FileCheck -check-prefix=COUNTONLY -input-file=%T/out %s \<br>
>> -// RUN:         && diff %t.cpp %t.base<br>
>> +// RUN: loop-convert . %t.cpp -- -I %S/Inputs<br>
>> +// RUN: FileCheck -input-file=%t.cpp %s<br>
>> +// RUN: cp %t.base %t.cpp<br>
>> +// RUN: loop-convert -count-only . %t.cpp -- -I %S/Inputs > %T/out<br>
>> +// RUN: FileCheck -check-prefix=COUNTONLY -input-file=%T/out %s<br>
>> +// RUN: diff %t.cpp %t.base<br>
>><br>
>>  #include "structures.h"<br>
>><br>
>><br>
>> Modified: clang-tools-extra/trunk/test/loop-convert/confidence.cpp<br>
>> URL: <a href="http://llvm.org/viewvc/llvm-project/clang-tools-extra/trunk/test/loop-convert/confidence.cpp?rev=163009&r1=163008&r2=163009&view=diff" target="_blank" class="cremed">http://llvm.org/viewvc/llvm-project/clang-tools-extra/trunk/test/loop-convert/confidence.cpp?rev=163009&r1=163008&r2=163009&view=diff</a><br>

>> ==============================================================================<br>
>> --- clang-tools-extra/trunk/test/loop-convert/confidence.cpp (original)<br>
>> +++ clang-tools-extra/trunk/test/loop-convert/confidence.cpp Fri Aug 31 12:49:33 2012<br>
>> @@ -1,9 +1,8 @@<br>
>> -// RUN: rm -rf %t.cpp<br>
>> -// RUN: grep -Ev "//\s*[A-Z-]+:" %s > %t.cpp<br>
>> -// RUN: loop-convert . %t.cpp -- -I %S/Inputs \<br>
>> -// RUN:         && FileCheck -input-file=%t.cpp %s<br>
>> -// RUN: loop-convert . %t.cpp -A2 -- -I %S/Inputs \<br>
>> -// RUN:         && FileCheck -check-prefix=RISKY -input-file=%t.cpp %s<br>
>> +// RUN: grep -Ev "// *[A-Z-]+:" %s > %t.cpp<br>
>> +// RUN: loop-convert . %t.cpp -- -I %S/Inputs<br>
>> +// RUN: FileCheck -input-file=%t.cpp %s<br>
>> +// RUN: loop-convert . %t.cpp -A2 -- -I %S/Inputs<br>
>> +// RUN: FileCheck -check-prefix=RISKY -input-file=%t.cpp %s<br>
>><br>
>>  #include "structures.h"<br>
>><br>
>><br>
>> Modified: clang-tools-extra/trunk/test/loop-convert/dependency.cpp<br>
>> URL: <a href="http://llvm.org/viewvc/llvm-project/clang-tools-extra/trunk/test/loop-convert/dependency.cpp?rev=163009&r1=163008&r2=163009&view=diff" target="_blank" class="cremed">http://llvm.org/viewvc/llvm-project/clang-tools-extra/trunk/test/loop-convert/dependency.cpp?rev=163009&r1=163008&r2=163009&view=diff</a><br>

>> ==============================================================================<br>
>> --- clang-tools-extra/trunk/test/loop-convert/dependency.cpp (original)<br>
>> +++ clang-tools-extra/trunk/test/loop-convert/dependency.cpp Fri Aug 31 12:49:33 2012<br>
>> @@ -1,5 +1,4 @@<br>
>> -// RUN: rm -rf %t.cpp<br>
>> -// RUN: grep -Ev "//\s*[A-Z-]+:" %s > %t.cpp<br>
>> +// RUN: grep -Ev "// *[A-Z-]+:" %s > %t.cpp<br>
>>  // RUN: loop-convert . %t.cpp -- && FileCheck -input-file=%t.cpp %s<br>
>><br>
>>  void f() {<br>
>><br>
>> Modified: clang-tools-extra/trunk/test/loop-convert/iterator.cpp<br>
>> URL: <a href="http://llvm.org/viewvc/llvm-project/clang-tools-extra/trunk/test/loop-convert/iterator.cpp?rev=163009&r1=163008&r2=163009&view=diff" target="_blank" class="cremed">http://llvm.org/viewvc/llvm-project/clang-tools-extra/trunk/test/loop-convert/iterator.cpp?rev=163009&r1=163008&r2=163009&view=diff</a><br>

>> ==============================================================================<br>
>> --- clang-tools-extra/trunk/test/loop-convert/iterator.cpp (original)<br>
>> +++ clang-tools-extra/trunk/test/loop-convert/iterator.cpp Fri Aug 31 12:49:33 2012<br>
>> @@ -1,9 +1,6 @@<br>
>> -// RUN: rm -rf %t.cpp<br>
>> -// RUN: grep -Ev "//\s*[A-Z-]+:" %s > %t.cpp<br>
>> -// RUN: loop-convert . %t.cpp -- -I %S/Inputs \<br>
>> -// RUN:           && FileCheck -input-file=%t.cpp %s<br>
>> -// RUN: grep -Ev "//\s*[A-Z-]+:" %s > %t.cpp<br>
>> -// RUN: rm -rf %t.cpp<br>
>> +// RUN: grep -Ev "// *[A-Z-]+:" %s > %t.cpp<br>
>> +// RUN: loop-convert . %t.cpp -- -I %S/Inputs<br>
>> +// RUN: FileCheck -input-file=%t.cpp %s<br>
>><br>
>>  #include "structures.h"<br>
>><br>
>><br>
>> Modified: clang-tools-extra/trunk/test/loop-convert/naming.cpp<br>
>> URL: <a href="http://llvm.org/viewvc/llvm-project/clang-tools-extra/trunk/test/loop-convert/naming.cpp?rev=163009&r1=163008&r2=163009&view=diff" target="_blank" class="cremed">http://llvm.org/viewvc/llvm-project/clang-tools-extra/trunk/test/loop-convert/naming.cpp?rev=163009&r1=163008&r2=163009&view=diff</a><br>

>> ==============================================================================<br>
>> --- clang-tools-extra/trunk/test/loop-convert/naming.cpp (original)<br>
>> +++ clang-tools-extra/trunk/test/loop-convert/naming.cpp Fri Aug 31 12:49:33 2012<br>
>> @@ -1,7 +1,6 @@<br>
>> -// RUN: rm -rf %t.cpp<br>
>> -// RUN: grep -Ev "//\s*[A-Z-]+:" %s > %t.cpp<br>
>> -// RUN: loop-convert . %t.cpp -- -I %S/Inputs \<br>
>> -// RUN:           && FileCheck -input-file=%t.cpp %s<br>
>> +// RUN: grep -Ev "// *[A-Z-]+:" %s > %t.cpp<br>
>> +// RUN: loop-convert . %t.cpp -- -I %S/Inputs<br>
>> +// RUN: FileCheck -input-file=%t.cpp %s<br>
>><br>
>>  #include "structures.h"<br>
>><br>
>><br>
>> Modified: clang-tools-extra/trunk/test/loop-convert/negative-iterator.cpp<br>
>> URL: <a href="http://llvm.org/viewvc/llvm-project/clang-tools-extra/trunk/test/loop-convert/negative-iterator.cpp?rev=163009&r1=163008&r2=163009&view=diff" target="_blank" class="cremed">http://llvm.org/viewvc/llvm-project/clang-tools-extra/trunk/test/loop-convert/negative-iterator.cpp?rev=163009&r1=163008&r2=163009&view=diff</a><br>

>> ==============================================================================<br>
>> --- clang-tools-extra/trunk/test/loop-convert/negative-iterator.cpp (original)<br>
>> +++ clang-tools-extra/trunk/test/loop-convert/negative-iterator.cpp Fri Aug 31 12:49:33 2012<br>
>> @@ -1,7 +1,6 @@<br>
>> -// RUN: rm -rf %t.cpp<br>
>> -// RUN: grep -Ev "//\s*[A-Z-]+:" %s > %t.cpp<br>
>> -// RUN: loop-convert . %t.cpp -- -I %S/Inputs \<br>
>> -// RUN:         && FileCheck -input-file=%t.cpp %s<br>
>> +// RUN: grep -Ev "// *[A-Z-]+:" %s > %t.cpp<br>
>> +// RUN: loop-convert . %t.cpp -- -I %S/Inputs<br>
>> +// RUN: FileCheck -input-file=%t.cpp %s<br>
>><br>
>>  #include "structures.h"<br>
>><br>
>><br>
>> Modified: clang-tools-extra/trunk/test/loop-convert/negative-multi-end-call.cpp<br>
>> URL: <a href="http://llvm.org/viewvc/llvm-project/clang-tools-extra/trunk/test/loop-convert/negative-multi-end-call.cpp?rev=163009&r1=163008&r2=163009&view=diff" target="_blank" class="cremed">http://llvm.org/viewvc/llvm-project/clang-tools-extra/trunk/test/loop-convert/negative-multi-end-call.cpp?rev=163009&r1=163008&r2=163009&view=diff</a><br>

>> ==============================================================================<br>
>> --- clang-tools-extra/trunk/test/loop-convert/negative-multi-end-call.cpp (original)<br>
>> +++ clang-tools-extra/trunk/test/loop-convert/negative-multi-end-call.cpp Fri Aug 31 12:49:33 2012<br>
>> @@ -1,7 +1,6 @@<br>
>> -// RUN: rm -rf %t.cpp<br>
>> -// RUN: grep -Ev "//\s*[A-Z-]+:" %s > %t.cpp<br>
>> -// RUN: loop-convert -A0 . %t.cpp -- -I %S/Inputs \<br>
>> -// RUN:         && FileCheck -input-file=%t.cpp %s<br>
>> +// RUN: grep -Ev "// *[A-Z-]+:" %s > %t.cpp<br>
>> +// RUN: loop-convert -A0 . %t.cpp -- -I %S/Inputs<br>
>> +// RUN: FileCheck -input-file=%t.cpp %s<br>
>><br>
>>  #include "structures.h"<br>
>><br>
>><br>
>> Modified: clang-tools-extra/trunk/test/loop-convert/negative-pseudoarray-extra.cpp<br>
>> URL: <a href="http://llvm.org/viewvc/llvm-project/clang-tools-extra/trunk/test/loop-convert/negative-pseudoarray-extra.cpp?rev=163009&r1=163008&r2=163009&view=diff" target="_blank" class="cremed">http://llvm.org/viewvc/llvm-project/clang-tools-extra/trunk/test/loop-convert/negative-pseudoarray-extra.cpp?rev=163009&r1=163008&r2=163009&view=diff</a><br>

>> ==============================================================================<br>
>> --- clang-tools-extra/trunk/test/loop-convert/negative-pseudoarray-extra.cpp (original)<br>
>> +++ clang-tools-extra/trunk/test/loop-convert/negative-pseudoarray-extra.cpp Fri Aug 31 12:49:33 2012<br>
>> @@ -1,7 +1,6 @@<br>
>> -// RUN: rm -rf %t.cpp<br>
>> -// RUN: grep -Ev "//\s*[A-Z-]+:" %s > %t.cpp<br>
>> -// RUN: loop-convert -A1 . %t.cpp -- -I %S/Inputs \<br>
>> -// RUN:         && FileCheck -input-file=%t.cpp %s<br>
>> +// RUN: grep -Ev "// *[A-Z-]+:" %s > %t.cpp<br>
>> +// RUN: loop-convert -A1 . %t.cpp -- -I %S/Inputs<br>
>> +// RUN: FileCheck -input-file=%t.cpp %s<br>
>><br>
>>  #include "structures.h"<br>
>><br>
>><br>
>> Modified: clang-tools-extra/trunk/test/loop-convert/negative-pseudoarray.cpp<br>
>> URL: <a href="http://llvm.org/viewvc/llvm-project/clang-tools-extra/trunk/test/loop-convert/negative-pseudoarray.cpp?rev=163009&r1=163008&r2=163009&view=diff" target="_blank" class="cremed">http://llvm.org/viewvc/llvm-project/clang-tools-extra/trunk/test/loop-convert/negative-pseudoarray.cpp?rev=163009&r1=163008&r2=163009&view=diff</a><br>

>> ==============================================================================<br>
>> --- clang-tools-extra/trunk/test/loop-convert/negative-pseudoarray.cpp (original)<br>
>> +++ clang-tools-extra/trunk/test/loop-convert/negative-pseudoarray.cpp Fri Aug 31 12:49:33 2012<br>
>> @@ -1,7 +1,6 @@<br>
>> -// RUN: rm -rf %t.cpp<br>
>> -// RUN: grep -Ev "//\s*[A-Z-]+:" %s > %t.cpp<br>
>> -// RUN: loop-convert -A1 . %t.cpp -- -I %S/Inputs \<br>
>> -// RUN:         && FileCheck -input-file=%t.cpp %s<br>
>> +// RUN: grep -Ev "// *[A-Z-]+:" %s > %t.cpp<br>
>> +// RUN: loop-convert -A1 . %t.cpp -- -I %S/Inputs<br>
>> +// RUN: FileCheck -input-file=%t.cpp %s<br>
>><br>
>>  #include "structures.h"<br>
>><br>
>><br>
>> Modified: clang-tools-extra/trunk/test/loop-convert/negative.cpp<br>
>> URL: <a href="http://llvm.org/viewvc/llvm-project/clang-tools-extra/trunk/test/loop-convert/negative.cpp?rev=163009&r1=163008&r2=163009&view=diff" target="_blank" class="cremed">http://llvm.org/viewvc/llvm-project/clang-tools-extra/trunk/test/loop-convert/negative.cpp?rev=163009&r1=163008&r2=163009&view=diff</a><br>

>> ==============================================================================<br>
>> --- clang-tools-extra/trunk/test/loop-convert/negative.cpp (original)<br>
>> +++ clang-tools-extra/trunk/test/loop-convert/negative.cpp Fri Aug 31 12:49:33 2012<br>
>> @@ -1,11 +1,9 @@<br>
>> -// RUN: rm -rf %t.cpp<br>
>> -// RUN: grep -Ev "//\s*[A-Z-]+:" %s > %t.cpp<br>
>> -// RUN: grep -Ev "//\s*[A-Z-]+:" %S/Inputs/negative-header.h > \<br>
>> +// RUN: grep -Ev "// *[A-Z-]+:" %s > %t.cpp<br>
>> +// RUN: grep -Ev "// *[A-Z-]+:" %S/Inputs/negative-header.h > \<br>
>>  // RUN:       %T/negative-header.h<br>
>> -// RUN: loop-convert . %t.cpp -- -I %S/Inputs/ \<br>
>> -// RUN:   && FileCheck -input-file=%t.cpp %s \<br>
>> -// RUN:   && FileCheck -input-file=%T/negative-header.h \<br>
>> -// RUN:                %S/Inputs/negative-header.h<br>
>> +// RUN: loop-convert . %t.cpp -- -I %S/Inputs/<br>
>> +// RUN: FileCheck -input-file=%t.cpp %s<br>
>> +// RUN: FileCheck -input-file=%T/negative-header.h %S/Inputs/negative-header.h<br>
>><br>
>>  #include "negative-header.h"<br>
>>  #include "structures.h"<br>
>><br>
>> Modified: clang-tools-extra/trunk/test/loop-convert/nesting.cpp<br>
>> URL: <a href="http://llvm.org/viewvc/llvm-project/clang-tools-extra/trunk/test/loop-convert/nesting.cpp?rev=163009&r1=163008&r2=163009&view=diff" target="_blank" class="cremed">http://llvm.org/viewvc/llvm-project/clang-tools-extra/trunk/test/loop-convert/nesting.cpp?rev=163009&r1=163008&r2=163009&view=diff</a><br>

>> ==============================================================================<br>
>> --- clang-tools-extra/trunk/test/loop-convert/nesting.cpp (original)<br>
>> +++ clang-tools-extra/trunk/test/loop-convert/nesting.cpp Fri Aug 31 12:49:33 2012<br>
>> @@ -1,7 +1,6 @@<br>
>> -// RUN: rm -rf %t.cpp<br>
>> -// RUN: grep -Ev "//\s*[A-Z-]+:" %s > %t.cpp<br>
>> -// RUN: loop-convert . %t.cpp -- -I %S/Inputs \<br>
>> -// RUN:           && FileCheck -input-file=%t.cpp %s<br>
>> +// RUN: grep -Ev "// *[A-Z-]+:" %s > %t.cpp<br>
>> +// RUN: loop-convert . %t.cpp -- -I %S/Inputs<br>
>> +// RUN: FileCheck -input-file=%t.cpp %s<br>
>><br>
>>  #include "structures.h"<br>
>><br>
>><br>
>> Modified: clang-tools-extra/trunk/test/loop-convert/nocompile.cpp<br>
>> URL: <a href="http://llvm.org/viewvc/llvm-project/clang-tools-extra/trunk/test/loop-convert/nocompile.cpp?rev=163009&r1=163008&r2=163009&view=diff" target="_blank" class="cremed">http://llvm.org/viewvc/llvm-project/clang-tools-extra/trunk/test/loop-convert/nocompile.cpp?rev=163009&r1=163008&r2=163009&view=diff</a><br>

>> ==============================================================================<br>
>> --- clang-tools-extra/trunk/test/loop-convert/nocompile.cpp (original)<br>
>> +++ clang-tools-extra/trunk/test/loop-convert/nocompile.cpp Fri Aug 31 12:49:33 2012<br>
>> @@ -1,8 +1,6 @@<br>
>> -// RUN: rm -rf %t.cpp<br>
>> -// RUN: grep -Ev "//\s*[A-Z-]+:" %s > %t.cpp<br>
>> -// RUN: loop-convert . %t.cpp -- -I %S/Inputs \<br>
>> -// RUN:         || FileCheck -input-file=%t.cpp %s<br>
>> -// Note that this test expects the compilation to fail!<br>
>> +// RUN: grep -Ev "// *[A-Z-]+:" %s > %t.cpp<br>
>> +// RUN: not loop-convert . %t.cpp -- -I %S/Inputs<br>
>> +// RUN: FileCheck -input-file=%t.cpp %s<br>
>><br>
>>  void valid() {<br>
>>    const int arr[5];<br>
>><br>
>> Modified: clang-tools-extra/trunk/test/loop-convert/pseudoarray.cpp<br>
>> URL: <a href="http://llvm.org/viewvc/llvm-project/clang-tools-extra/trunk/test/loop-convert/pseudoarray.cpp?rev=163009&r1=163008&r2=163009&view=diff" target="_blank" class="cremed">http://llvm.org/viewvc/llvm-project/clang-tools-extra/trunk/test/loop-convert/pseudoarray.cpp?rev=163009&r1=163008&r2=163009&view=diff</a><br>

>> ==============================================================================<br>
>> --- clang-tools-extra/trunk/test/loop-convert/pseudoarray.cpp (original)<br>
>> +++ clang-tools-extra/trunk/test/loop-convert/pseudoarray.cpp Fri Aug 31 12:49:33 2012<br>
>> @@ -1,8 +1,6 @@<br>
>> -// RUN: rm -rf %t.cpp<br>
>> -// RUN: grep -Ev "//\s*[A-Z-]+:" %s > %t.cpp<br>
>> -// RUN: loop-convert . %t.cpp -- -I %S/Inputs \<br>
>> -// RUN:           && FileCheck -input-file=%t.cpp %s<br>
>> -// RUN: rm -rf %t.cpp<br>
>> +// RUN: grep -Ev "// *[A-Z-]+:" %s > %t.cpp<br>
>> +// RUN: loop-convert . %t.cpp -- -I %S/Inputs<br>
>> +// RUN: FileCheck -input-file=%t.cpp %s<br>
>>  #include "structures.h"<br>
>><br>
>>  const int N = 6;<br>
>><br>
>> Modified: clang-tools-extra/trunk/test/loop-convert/single-iterator.cpp<br>
>> URL: <a href="http://llvm.org/viewvc/llvm-project/clang-tools-extra/trunk/test/loop-convert/single-iterator.cpp?rev=163009&r1=163008&r2=163009&view=diff" target="_blank" class="cremed">http://llvm.org/viewvc/llvm-project/clang-tools-extra/trunk/test/loop-convert/single-iterator.cpp?rev=163009&r1=163008&r2=163009&view=diff</a><br>

>> ==============================================================================<br>
>> --- clang-tools-extra/trunk/test/loop-convert/single-iterator.cpp (original)<br>
>> +++ clang-tools-extra/trunk/test/loop-convert/single-iterator.cpp Fri Aug 31 12:49:33 2012<br>
>> @@ -1,9 +1,6 @@<br>
>> -// RUN: rm -rf %t.cpp<br>
>> -// RUN: grep -Ev "//\s*[A-Z-]+:" %s > %t.cpp<br>
>> -// RUN: loop-convert . %t.cpp -- -I %S/Inputs \<br>
>> -// RUN:           && FileCheck -input-file=%t.cpp %s<br>
>> -// RUN: grep -Ev "//\s*[A-Z-]+:" %s > %t.cpp<br>
>> -// RUN: rm -rf %t.cpp<br>
>> +// RUN: grep -Ev "// *[A-Z-]+:" %s > %t.cpp<br>
>> +// RUN: loop-convert . %t.cpp -- -I %S/Inputs<br>
>> +// RUN: FileCheck -input-file=%t.cpp %s<br>
>><br>
>>  #include "structures.h"<br>
>><br>
>><br>
>> Modified: clang-tools-extra/trunk/test/remove-cstr-calls/basic.cpp<br>
>> URL: <a href="http://llvm.org/viewvc/llvm-project/clang-tools-extra/trunk/test/remove-cstr-calls/basic.cpp?rev=163009&r1=163008&r2=163009&view=diff" target="_blank" class="cremed">http://llvm.org/viewvc/llvm-project/clang-tools-extra/trunk/test/remove-cstr-calls/basic.cpp?rev=163009&r1=163008&r2=163009&view=diff</a><br>

>> ==============================================================================<br>
>> --- clang-tools-extra/trunk/test/remove-cstr-calls/basic.cpp (original)<br>
>> +++ clang-tools-extra/trunk/test/remove-cstr-calls/basic.cpp Fri Aug 31 12:49:33 2012<br>
>> @@ -1,7 +1,9 @@<br>
>> -// RUN: echo '[{"directory":".","command":"clang++ -c %T/test.cpp","file":"%T/test.cpp"}]' > %T/compile_commands.json<br>
>> -// RUN: cp "%s" "%T/test.cpp"<br>
>> -// RUN: remove-cstr-calls "%T" "%T/test.cpp"<br>
>> -// RUN: cat "%T/test.cpp" | FileCheck %s<br>
>> +// RUN: rm -rf %t<br>
>> +// RUN: mkdir %t<br>
>> +// RUN: echo "[{\"directory\":\".\",\"command\":\"clang++ -c %t/test.cpp\",\"file\":\"%t/test.cpp\"}]" | sed -e 's/\\/\//g' > %t/compile_commands.json<br>

>> +// RUN: cp "%s" "%t/test.cpp"<br>
>> +// RUN: remove-cstr-calls "%t" "%t/test.cpp"<br>
>> +// RUN: cat "%t/test.cpp" | FileCheck %s<br>
>>  // REQUIRES: shell<br>
>>  // FIXME: implement a mode for refactoring tools that takes input from stdin<br>
>>  // and writes output to stdout for easier testing of tools.<br>
>><br>
>><br>
>> _______________________________________________<br>
>> cfe-commits mailing list<br>
>> <a href="mailto:cfe-commits@cs.uiuc.edu" class="cremed">cfe-commits@cs.uiuc.edu</a><br>
>> <a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits" target="_blank" class="cremed">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits</a><br>
_______________________________________________<br>
cfe-commits mailing list<br>
<a href="mailto:cfe-commits@cs.uiuc.edu" class="cremed">cfe-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits" target="_blank" class="cremed">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits</a><br>
</div></div></blockquote></div><br></div>