[PATCH] D28896: [FileCheck] Add directive for checking for blank lines

James Henderson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jun 26 07:48:55 PDT 2018


jhenderson added inline comments.


================
Comment at: docs/CommandGuide/FileCheck.rst:247
+
+If you need to check that the next line is blank, you can use the
+"``CHECK-EMPTY:``" directive.
----------------
probinson wrote:
> Hm.  Is whitespace allowed on the "blank" line?  Even if `--match-full-lines` is not in effect?  If whitespace is always disallowed, then maybe instead of "is blank" this should say something like "has nothing on it, not even whitespace" ?
At the moment, whitespace isn't allowed (see case 6 in the test). This could probably be changed, if there's call for it, but I figured that it was simpler this way.

I'll change the wording as suggested.


Repository:
  rL LLVM

https://reviews.llvm.org/D28896





More information about the llvm-commits mailing list