[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