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