================ 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 _______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
