[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