[PATCH][RFC] Add 'CHECK-<N>' directive in FileCheck

Michael Liao michael.liao at intel.com
Mon May 13 20:45:06 PDT 2013


I though the latest patch already changes that syntax, CHECK-<N>, and
includes angle brackets as part of that directive. In case those patches
are missing, I re-attached. One is core refactoring and the other adds
'CHECK-<N>'.

- Michael

On Mon, 2013-05-13 at 19:15 -0700, Richard Smith wrote:
> If we keep this feature, please find some other syntax for it.
> 
> 
>    grep 'CHECK-[0-9]' tools/clang/test
> 
> 
> has *many* hits which would be broken by this.
> 
> On Mon, May 13, 2013 at 4:11 PM, Chandler Carruth
> <chandlerc at google.com> wrote:
>         On Tue, May 14, 2013 at 1:07 AM, Jakob Stoklund Olesen
>         <stoklund at 2pi.dk> wrote:
>         
>                 I think we should leave ‘grep’ in those tests until
>                 somebody properly fixes them. I don’t think there is
>                 any benefit to mechanically changing them to use
>                 FileCheck instead.
>         
>         
>         Not really advocating strongly for this feature, but it does
>         have some value: it removes our test suite's reliance on the
>         'grep' tool on the host system, oddities of the syntax it
>         uses, etc etc... Not sure its enough to justify the effort,
>         but it is something.
>         
>         _______________________________________________
>         llvm-commits mailing list
>         llvm-commits at cs.uiuc.edu
>         http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits
>         
> 
> 

-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0002-Add-CHECK-N-support.patch
Type: text/x-patch
Size: 8129 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20130513/08df307d/attachment.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-Refactor-string-checking.patch
Type: text/x-patch
Size: 7776 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20130513/08df307d/attachment-0001.bin>


More information about the llvm-commits mailing list