[PATCH] Add support to FileCheck for out-of-order matching
Michael Liao
michael.liao at intel.com
Wed Apr 24 14:37:30 PDT 2013
On Wed, 2013-04-24 at 14:32 -0700, Eli Bendersky wrote:
> On Wed, Apr 24, 2013 at 11:35 AM, Michael Liao
> <michael.liao at intel.com> wrote:
> Here's the revised patch achieving that goal. When a match is
> deferred,
> we will check whethe that check string has DAG strings and
> match them
> first. If all of them could match, we will match that check
> string again
> starting from the end of DAG string matchings. I kept the
> logic checking
> DAG string once a check string itself could match as it will
> reduce the
> range to verify for DAG strings. How do you think that? I
> could remove
> it if we think it's unnecessary.
>
> There's one feature might be useful. (I disabled it in both
> patches
> now.) Is it valueable to mix CHECK-NOT and CHECK-DAG together?
>
>
> What I don't see in this patch is updated documentation. Personally
> I'm a bit worried about the growing complexity of FileCheck logic.
> Some tests are non-trivial to follow even without the new features
> (specifically I mean when they fail and one needs to try to follow
> FileCheck's matching path in the result file). Those can make some
> tests, if sloppily written, even more incomprehensible. Whatever is
> decided, the new behavior has to be very well defined and documented
> (with examples). This will hopefully lower the chance people write
> overly-complex tests.
Yeah, totally agree. Document will be updated once we nail down the
exact behavior of CHECK-DAG.
- michael
>
>
> Eli
>
>
>
More information about the llvm-commits
mailing list