[cfe-commits] [PATCH] Support for [[@LINE]], [[@LINE+<offset>]], [[@LINE-<offset>]] expressions in FileCheck

Alexander Kornienko alexfh at google.com
Wed Nov 14 04:46:37 PST 2012



================
Comment at: docs/CommandGuide/FileCheck.rst:295
@@ +294,3 @@
+~~~~~~~~~~~~~~~~~~~~~
+
+
----------------
Dmitri Gribenko wrote:
> Are two empty lines required?
No idea, but the whole file uses two empty lines before and after headers.

================
Comment at: docs/CommandGuide/FileCheck.rst:303
@@ +302,3 @@
+
+To support this case, FileCheck allows using **[[@LINE]]**,
+**[[@LINE+<offset>]]**, **[[@LINE-<offset>]]** expressions in patterns. These
----------------
Dmitri Gribenko wrote:
> Maybe use double backticks to mark code?  Like ```this```.
I use the same style as in the rest of the file (see line #286, for example).

================
Comment at: utils/FileCheck/FileCheck.cpp:112
@@ +111,3 @@
+
+  /// \brief Evaluates expression and stores the result to Value.
+  /// \return true on success. false when the expression has invalid syntax.
----------------
Dmitri Gribenko wrote:
> ... to \p Value.
Done.

================
Comment at: utils/FileCheck/FileCheck.cpp:208
@@ +207,3 @@
+      // supports @LINE, @LINE+number, @LINE-number expressions. The check here
+      // is relaxed, more strickt check is performed in EvaluateExpression.
+      bool IsExpression = false;
----------------
Dmitri Gribenko wrote:
> sp: strict
> ... in \c EvaluateExpression.
Oops, typo ;)
Done.


http://llvm-reviews.chandlerc.com/D109



More information about the cfe-commits mailing list