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

Alexander Kornienko alexfh at google.com
Tue Nov 13 15:51:33 PST 2012


  Please take another look.


================
Comment at: utils/FileCheck/FileCheck.cpp:112
@@ -107,1 +111,3 @@
+
+  bool EvaluateExpression(StringRef Expr, std::string &Value) const;
 };
----------------
Dmitri Gribenko wrote:
> Please document the return value (true is success).
Added.

================
Comment at: utils/FileCheck/FileCheck.cpp:68
@@ +67,3 @@
+  /// \brief Contains the number of line this pattern is in.
+  int LineNumber;
+
----------------
Dmitri Gribenko wrote:
> Line numbers are usually stored as 'unsigned'.  (Here and a few cases below.)
Done.

================
Comment at: utils/FileCheck/FileCheck.cpp:204
@@ -195,2 +203,3 @@
 
       // Verify that the name is well formed.
+      bool IsExpression = false;
----------------
Dmitri Gribenko wrote:
> This makes 'Name' a bit of a misnomer...
> 
> Please expand comment to include that name can also be an expression of the form @foo, @foo+something, @foo-something.
Added some comments here.


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



More information about the llvm-commits mailing list