[PATCH] add ConstEvaluatedExprVisitor

Richard Smith richard at metafoo.co.uk
Fri Jun 5 12:17:02 PDT 2015


LGTM


================
Comment at: llvm/tools/clang/include/clang/AST/EvaluatedExprVisitor.h:34
@@ -33,3 +33,3 @@
 public:
-  explicit EvaluatedExprVisitor(ASTContext &Context) : Context(Context) { }
-  
+
+#define PTR(CLASS) typename Ptr<CLASS>::type
----------------
No blank line here.

================
Comment at: llvm/tools/clang/include/clang/AST/EvaluatedExprVisitor.h:114
@@ +113,3 @@
+public:
+
+  explicit EvaluatedExprVisitor(const ASTContext &Context) :
----------------
Likewise.

================
Comment at: llvm/tools/clang/include/clang/AST/EvaluatedExprVisitor.h:124
@@ +123,3 @@
+public:
+
+  explicit ConstEvaluatedExprVisitor(const ASTContext &Context) :
----------------
Likewise.

http://reviews.llvm.org/D10210

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/






More information about the cfe-commits mailing list