This is an archive of the discontinued LLVM Phabricator instance.

[MLIR, test] Fix use of undef FileCheck var
ClosedPublic

Authored by thopre on Apr 6 2021, 7:25 AM.

Details

Summary

MLIR test Dialect/Linalg/transform-patterns.mlir tries to check for the
absence of a sequence of instructions with several CHECK-NOT with one of
those directives using a variable defined in another. However CHECK-NOT
are checked independently so that is using a variable defined in a
pattern that should not occur in the input.

This commit removes the dependency between those CHECK-NOT by replacing
occurences of variables by the regex that were used to define them.

Note to reviewers: please pay attention to whether the remaining
reference to l0 is correct. There was a l0 defined in those CHECK-NOT
and one defined before. I'm not sure what was the intent there.

Diff Detail

Event Timeline

thopre created this revision.Apr 6 2021, 7:25 AM
thopre requested review of this revision.Apr 6 2021, 7:25 AM
Herald added a project: Restricted Project. · View Herald TranscriptApr 6 2021, 7:25 AM
This revision is now accepted and ready to land.Apr 6 2021, 8:38 AM
thopre added a comment.Apr 6 2021, 8:45 AM

thanks!

Is the line 319 referencing the right version of l0?? Is there an equivalent to update_test_checks for mlir-opt that we could use perhaps?

thanks!

Is the line 319 referencing the right version of l0?? Is there an equivalent to update_test_checks for mlir-opt that we could use perhaps?

@poechsel ^^ Is the intent of line 319 to refer to l0 as defined by line 312?

Yes! The intent is indeed to refer to the l0 defined on line 312

This revision was automatically updated to reflect the committed changes.