adding statistics about how many different kinds of expressions we have created in...

this is useful, for example, with --parse-only, to figure out a bit of problem structure

diff --git a/src/expr/expr_manager_template.cpp b/src/expr/expr_manager_template.cpp
index c59be7749aa2d7629cad14944acc229ec2bca9d7..3769d47d2a700ec01c1f51fd14b954987c9e7140 100644 (file)
--- a/src/expr/expr_manager_template.cpp
+++ b/src/expr/expr_manager_template.cpp
@@ -20,6 +20,7 @@
 #include "expr/expr_manager.h"
 #include "context/context.h"
 #include "util/options.h"
+#include "util/stats.h"
 
 ${includes}
 
@@ -29,6 +30,21 @@ ${includes}
 // since it'll get overwritten on a later build.
 #line 31 "${template}"
 
+#ifdef CVC4_STATISTICS_ON
+  #define INC_STAT(kind) \
+  { \
+    if (d_exprStatistics[kind] == NULL) { \
+      stringstream statName; \
+      statName << "CVC4::expr::count" << kind; \
+      d_exprStatistics[kind] = new IntStat(statName.str(), 0); \
+      StatisticsRegistry::registerStat(d_exprStatistics[kind]); \
+    } \
+    ++ *(d_exprStatistics[kind]); \
+  } 
+#else 
+  #define INC_STAT(kind)
+#endif
+
 using namespace std;
 using namespace CVC4::context;
 using namespace CVC4::kind;
@@ -38,16 +54,34 @@ namespace CVC4 {
 ExprManager::ExprManager() :
   d_ctxt(new Context),
   d_nodeManager(new NodeManager(d_ctxt)) {
+#ifdef CVC4_STATISTICS_ON   
+  for (unsigned i = 0; i < kind::LAST_KIND; ++ i) {
+    d_exprStatistics[i] = NULL;
+  }
+#endif
 }
 
 ExprManager::ExprManager(const Options& options) :
   d_ctxt(new Context),
   d_nodeManager(new NodeManager(d_ctxt, options)) {
+#ifdef CVC4_STATISTICS_ON   
+  for (unsigned i = 0; i < kind::LAST_KIND; ++ i) {
+    d_exprStatistics[i] = NULL;
+  }
+#endif
 }
 
 ExprManager::~ExprManager() {
   delete d_nodeManager;
   delete d_ctxt;
+#ifdef CVC4_STATISTICS_ON   
+  for (unsigned i = 0; i < kind::LAST_KIND; ++ i) {
+    if (d_exprStatistics[i] != NULL) {
+      StatisticsRegistry::unregisterStat(d_exprStatistics[i]);
+      delete d_exprStatistics[i];
+    }
+  }
+#endif
 }
 
 BooleanType ExprManager::booleanType() const {
@@ -71,7 +105,7 @@ IntegerType ExprManager::integerType() const {
 }
 
 Expr ExprManager::mkExpr(Kind kind, const Expr& child1) {
-  const unsigned n = 1;
+  const unsigned n = 1; 
   CheckArgument(n >= minArity(kind) && n <= maxArity(kind), kind,
                 "Exprs with kind %s must have at least %u children and "
                 "at most %u children (the one under construction has %u)",
@@ -79,6 +113,7 @@ Expr ExprManager::mkExpr(Kind kind, const Expr& child1) {
                 minArity(kind), maxArity(kind), n);
   NodeManagerScope nms(d_nodeManager);
   try {
+    INC_STAT(kind);
     return Expr(this, d_nodeManager->mkNodePtr(kind, child1.getNode()));
   } catch (const TypeCheckingExceptionPrivate& e) {
     throw TypeCheckingException(this, &e);
@@ -94,6 +129,7 @@ Expr ExprManager::mkExpr(Kind kind, const Expr& child1, const Expr& child2) {
                 minArity(kind), maxArity(kind), n);
   NodeManagerScope nms(d_nodeManager);
   try {
+    INC_STAT(kind);
     return Expr(this, d_nodeManager->mkNodePtr(kind,
                                                child1.getNode(),
                                                child2.getNode()));
@@ -112,6 +148,7 @@ Expr ExprManager::mkExpr(Kind kind, const Expr& child1, const Expr& child2,
                 minArity(kind), maxArity(kind), n);
   NodeManagerScope nms(d_nodeManager);
   try {
+    INC_STAT(kind);
     return Expr(this, d_nodeManager->mkNodePtr(kind,
                                                child1.getNode(),
                                                child2.getNode(),
@@ -131,6 +168,7 @@ Expr ExprManager::mkExpr(Kind kind, const Expr& child1, const Expr& child2,
                 minArity(kind), maxArity(kind), n);
   NodeManagerScope nms(d_nodeManager);
   try {
+    INC_STAT(kind);
     return Expr(this, d_nodeManager->mkNodePtr(kind,
                                                child1.getNode(),
                                                child2.getNode(),
@@ -152,6 +190,7 @@ Expr ExprManager::mkExpr(Kind kind, const Expr& child1, const Expr& child2,
                 minArity(kind), maxArity(kind), n);
   NodeManagerScope nms(d_nodeManager);
   try {
+    INC_STAT(kind);
     return Expr(this, d_nodeManager->mkNodePtr(kind,
                                                child1.getNode(),
                                                child2.getNode(),
@@ -181,6 +220,7 @@ Expr ExprManager::mkExpr(Kind kind, const std::vector<Expr>& children) {
     ++it;
   }
   try {
+    INC_STAT(kind);
     return Expr(this, d_nodeManager->mkNodePtr(kind, nodes));
   } catch (const TypeCheckingExceptionPrivate& e) {
     throw TypeCheckingException(this, &e);
@@ -206,6 +246,7 @@ Expr ExprManager::mkExpr(Expr opExpr, const std::vector<Expr>& children) {
     ++it;
   }
   try {
+    INC_STAT(kind);
     return Expr(this,d_nodeManager->mkNodePtr(opExpr.getNode(), nodes));
   } catch (const TypeCheckingExceptionPrivate& e) {
     throw TypeCheckingException(this, &e);

diff --git a/src/expr/expr_manager_template.h b/src/expr/expr_manager_template.h
index e5df3ced3999e0e6dba71bbe36701ea44a3f2039..1bb9fd9fd935225ab6a1a5a5c5d21121f05693d1 100644 (file)
--- a/src/expr/expr_manager_template.h
+++ b/src/expr/expr_manager_template.h
@@ -41,6 +41,7 @@ class Expr;
 class SmtEngine;
 class NodeManager;
 class Options;
+class IntStat;
 
 namespace context {
   class Context;
@@ -54,6 +55,9 @@ private:
   /** The internal node manager */
   NodeManager* d_nodeManager;
 
+  /** Counts of expressions created of a given kind */
+  IntStat* d_exprStatistics[kind::LAST_KIND];
+
   /**
    * Returns the internal node manager.  This should only be used by
    * internal users, i.e. the friend classes.