[PATCH] D60386: FileCheck [6/12]: Introduce numeric variable definition

Thomas Preud'homme via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue May 14 03:34:44 PDT 2019


thopre updated this revision to Diff 199390.
thopre marked 13 inline comments as done.
thopre added a comment.

- Address Paul's review comments
- Rename numeric expression subsitution for numeric substitution to clearly distinguish numeric expression (the part that gets evaluated) from the whole #<something> that gets substituted (which might include a numeric variable definition)
- Rename ParsePattern into parsePattern since it is modified


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D60386

Files:
  llvm/docs/CommandGuide/FileCheck.rst
  llvm/include/llvm/Support/FileCheck.h
  llvm/lib/Support/FileCheck.cpp
  llvm/test/FileCheck/numeric-defines-diagnostics.txt
  llvm/test/FileCheck/numeric-expression.txt
  llvm/test/FileCheck/var-scope.txt
  llvm/test/FileCheck/verbose.txt
  llvm/unittests/Support/FileCheckTest.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D60386.199390.patch
Type: text/x-patch
Size: 63353 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20190514/b23d1d2e/attachment-0001.bin>


More information about the llvm-commits mailing list