[PATCH] D66141: [FileCheck] Forbid using var defined on same line

Joel E. Denny via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Aug 16 06:53:00 PDT 2019


jdenny added inline comments.


================
Comment at: llvm/test/FileCheck/numeric-expression.txt:180
 DEF-EXPR-FAIL-NEXT: [[# VAR20:]]
 DEF-EXPR-FAIL-NEXT: [[# VAR42: VAR20+22]]
 DEF-EXPR-FAIL-MSG: numeric-expression.txt:[[#@LINE-1]]:21: error: {{D}}EF-EXPR-FAIL-NEXT: is not on the line after the previous match
----------------
thopre wrote:
> jdenny wrote:
> > After the earlier removal in this file, is this the only remaining check that you can use a numeric variable within another's pattern?  There should probably be a positive check too.  Sorry if I overlooked one.
> There is a positive check removed above (see lines 85-92 in the before pane).
That's the removal I was referring to.  So, unless I overlooked one, there's now no positive check, but shouldn't there be one?  For example:

```
20
42
CHECK: [[# VAR20:]]
CHECK-NEXT: [[# VAR42: VAR20+22]]
```


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D66141/new/

https://reviews.llvm.org/D66141





More information about the llvm-commits mailing list