From: trevor_hansen <trevor_hansen@e59a4935-1847-0410-ae03-e826735625c1>
Date: Mon, 23 Apr 2012 04:58:46 +0000 (+0000)
Subject: Improvments to code used for experiments.
X-Git-Url: https://git.unchartedbackwaters.co.uk/w/?a=commitdiff_plain;h=e10c60b46a88214ade56024a4046727a21977b66;p=francis%2Fstp.git

Improvments to code used for experiments.



git-svn-id: https://stp-fast-prover.svn.sourceforge.net/svnroot/stp-fast-prover/trunk/stp@1654 e59a4935-1847-0410-ae03-e826735625c1
---

diff --git a/src/util/BBAsProp.h b/src/util/BBAsProp.h
index fa341bd..b40ba03 100644
--- a/src/util/BBAsProp.h
+++ b/src/util/BBAsProp.h
@@ -5,6 +5,9 @@
 #ifndef BBASPROP_H_
 #define BBASPROP_H_
 
+#include "../sat/core/Solver.h"
+
+
 class BBAsProp
 {
 public:
@@ -91,13 +94,13 @@ public:
   }
 
   int
-  addToClauseCount()
+  fixedCount()
   {
     return addToClauseCount(i0) +addToClauseCount(i1) + addToClauseCount(r);
   }
 
   int
-  addToClauseCount( ASTNode n)
+  addToClauseCount(const ASTNode n)
   {
     int result =0;
     const int bits = std::max(1U, n.GetValueWidth());
diff --git a/src/util/Functions.h b/src/util/Functions.h
index 18ccefc..2d8f791 100644
--- a/src/util/Functions.h
+++ b/src/util/Functions.h
@@ -40,6 +40,76 @@ signedModulus(vector<FixedBits*>& children, FixedBits& output);
 Result
 unsignedModulus(vector<FixedBits*>& children, FixedBits& output);
 
+int bvOrF(int a, int b)
+{
+  return a | b;
+}
+
+int bvXOrF(int a, int b)
+{
+  return a ^ b;
+}
+
+int bvAndF(int a, int b)
+{
+  return a &b;
+}
+
+int rightSF(int a, int b)
+{
+  if (b >= sizeof(int)*8)
+    return 0;
+
+  return a>>b;
+}
+
+int leftSF(int a, int b)
+{
+  if (b >= sizeof(int)*8)
+    return 0;
+
+  return a<<b;
+}
+
+int plusF(int a, int b)
+{
+  return a+b;
+}
+
+int multiplyF(int a, int b)
+{
+  return a*b;
+}
+
+int divideF(int a, int b)
+{
+  if (b==0)
+    return 1;
+  return a/b;
+}
+
+
+int subF(int a, int b)
+{
+  return a-b;
+}
+
+int eqF(int a, int b)
+{
+  return (a==b)? 1:0;
+}
+
+int ltF(int a, int b)
+{
+  return (a<b)? 1:0;
+}
+
+int remF(int a, int b)
+{
+  if (b ==0)
+    return a;
+  return (a %b);
+}
 
 
 struct Functions
@@ -49,36 +119,49 @@ struct Functions
     Kind k;
     string name;
     Result (*fn)(vector<FixedBits*>&, FixedBits&);
+    int(*op)(int o1, int o2);
 
-    Function (Kind k_, string name_, Result (*fn_)(vector<FixedBits*>&, FixedBits&) )
+    Function (Kind k_, string name_, Result (*fn_)(vector<FixedBits*>&, FixedBits&), int(*op_)(int o1, int o2) )
     {
       name = name_;
       k = k_;
       fn = fn_;
+      op = op_;
     }
   };
 
   std::list<Function> l;
 
+  /*
+
+      exhaustively_check(i, BVRIGHTSHIFT, &bvRightShiftBothWays, &rightSF);
+      exhaustively_check(i, BVPLUS, &bvAddBothWays, &plusF);
+
+      exhaustively_check(i, BVAND, &bvAndBothWays, &bvandF);
+      exhaustively_check(i, BVOR, &bvOrBothWays, &bvorF);
+      }
+      */
 
   Functions()
   {
-    l.push_back(Function(BVSGE, "signed greater than equals", &bvSignedGreaterThanEqualsBothWays));
-    l.push_back(Function(BVLT, "unsigned less than", &bvLessThanEqualsBothWays));
-    l.push_back(Function(EQ, "equals", &bvEqualsBothWays));
-    l.push_back(Function(BVXOR, "bit-vector xor", &bvXorBothWays));
-    l.push_back(Function(BVOR, "bit-vector or", &bvOrBothWays));
-    l.push_back(Function(BVAND, "bit-vector and", &bvAndBothWays));
-    l.push_back(Function(BVRIGHTSHIFT, "right shift", &bvRightShiftBothWays));
-    l.push_back(Function(BVLEFTSHIFT, "left shift", &bvLeftShiftBothWays));
-    l.push_back(Function(BVSRSHIFT, "arithmetic shift", &bvArithmeticRightShiftBothWays));
-    l.push_back(Function(BVPLUS, "addition", &bvAddBothWays));
-    l.push_back(Function(BVMULT, "multiplication", &multiply));
-    l.push_back(Function(BVDIV, "unsigned division", &unsignedDivide));
-    l.push_back(Function(BVMOD, "unsigned remainder", &unsignedModulus));
-    l.push_back(Function(SBVDIV, "signed division", &signedDivide));
-    l.push_back(Function(SBVREM, "signed remainder",&signedRemainder ));
-}
+    //l.push_back(Function(BVSGE, "signed greater than equals", &bvSignedGreaterThanEqualsBothWays));
+    l.push_back(Function(BVLT, "unsigned less than", &bvLessThanBothWays, &ltF));
+    l.push_back(Function(EQ, "equals", &bvEqualsBothWays, &eqF));
+    l.push_back(Function(BVXOR, "bit-vector xor", &bvXorBothWays, &bvXOrF));
+    l.push_back(Function(BVOR, "bit-vector or", &bvOrBothWays, &bvOrF ));
+    l.push_back(Function(BVAND, "bit-vector and", &bvAndBothWays, &bvAndF));
+    l.push_back(Function(BVRIGHTSHIFT, "right shift", &bvRightShiftBothWays, &rightSF));
+    l.push_back(Function(BVLEFTSHIFT, "left shift", &bvLeftShiftBothWays, &leftSF));
+    //l.push_back(Function(BVSRSHIFT, "arithmetic shift", &bvArithmeticRightShiftBothWays));
+    l.push_back(Function(BVPLUS, "addition", &bvAddBothWays, &plusF));
+    l.push_back(Function(BVSUB, "subtraction", &bvSubtractBothWays, &subF));
+    l.push_back(Function(BVMULT, "multiplication", &multiply, &multiplyF));
+    l.push_back(Function(BVDIV, "unsigned division", &unsignedDivide, &divideF));
+    l.push_back(Function(BVMOD, "unsigned remainder", &unsignedModulus, &remF));
+    //l.push_back(Function(SBVDIV, "signed division", &signedDivide));
+    //l.push_back(Function(SBVREM, "signed remainder",&signedRemainder ));
+  }
+
 
 
 
diff --git a/src/util/test_cbitp.cpp b/src/util/test_cbitp.cpp
index 04ee963..e270115 100644
--- a/src/util/test_cbitp.cpp
+++ b/src/util/test_cbitp.cpp
@@ -23,6 +23,8 @@
 
 #include "StopWatch.h"
 #include "Relations.h"
+#include "BBAsProp.h"
+#include "Functions.h"
 
 using simplifier::constantBitP::FixedBits;
 using namespace simplifier::constantBitP;
@@ -311,22 +313,27 @@ struct D
 };
 
 void
-exhaustively_check(const int bitwidth, Result (*transfer)(vector<FixedBits*>&, FixedBits&), int (*op)(int a, int b))
+exhaustively_check(const int bitwidth, Kind k, Result (*transfer)(vector<FixedBits*>&, FixedBits&), int (*op)(int a, int b))
 {
   vector<D> list;
 
+  int transferBad =0;
+  int BBBad =0;
+
   const int numberOfInputParams = 2;
 
-  assert(numberOfInputParams >0);
-  const int mask = pow(2, bitwidth) - 1;
+  int resultLength = (is_Form_kind(k) ) ? 1:bitwidth;
+
+  const int input_mask = pow(2, bitwidth) - 1;
+  const int output_mask = resultLength ==1 ? 1: (pow(2, bitwidth)-1);
 
   // Create all the possible inputs, and apply the function.
   for (int i = 0; i < pow(2, bitwidth * numberOfInputParams); i++)
     {
     D d;
-    d.a = i & mask;
-    d.b = (i >> bitwidth) & mask;
-    d.c = op(d.a,d.b) & mask;
+    d.a = i & input_mask;
+    d.b = (i >> bitwidth) & input_mask;
+    d.c = op(d.a,d.b) & output_mask;
     list.push_back(d);
     }
 
@@ -335,7 +342,9 @@ exhaustively_check(const int bitwidth, Result (*transfer)(vector<FixedBits*>&, F
   vector<FixedBits*> temp;
   temp.push_back(&a);
   temp.push_back(&b);
-  FixedBits tempOutput(bitwidth, false);
+
+
+  FixedBits tempOutput(resultLength, false);
 
   vector<int> lengths;
   int totalLength = 0;
@@ -347,35 +356,37 @@ exhaustively_check(const int bitwidth, Result (*transfer)(vector<FixedBits*>&, F
     lengths.push_back(children[i]->getWidth());
     }
 
-  int resultLength = tempOutput.getWidth();
+
   FixedBits output(resultLength, tempOutput.isBoolean());
 
   lengths.push_back(resultLength);
   totalLength += resultLength;
 
   FixedBits empty(bitwidth, false);
-  FixedBits c_a(bitwidth, false), c_b(bitwidth, false), c_o(bitwidth, false);
+  FixedBits c_a(bitwidth, false), c_b(bitwidth, false), c_o(resultLength, false);
+
+  BBAsProp BBP(k,mgr,bitwidth);
 
   const int to_iterate = pow(3, totalLength);
   for (long j = 0; j < to_iterate; j++)
     {
     int current = j;
 
-    if (j% 100000 == 0)
-      cerr << j << " of " << to_iterate << endl;
+    //if (j% 100000 == 0)
+    //  cerr << j << " of " << to_iterate << endl;
 
     int id = 0;
     int usedUp = 0;
 
-    for (int k = 0; k < totalLength; k++)
+    for (int k_it = 0; k_it < totalLength; k_it++)
       {
-      if (k < resultLength)
+      if (k_it < resultLength)
         {
-        setV(output, k, current % 3);
+        setV(output, k_it, current % 3);
         }
       else
         {
-        int working = (k - resultLength - usedUp);
+        int working = (k_it - resultLength - usedUp);
         if (working == lengths[id])
           {
           usedUp += lengths[id];
@@ -387,18 +398,18 @@ exhaustively_check(const int bitwidth, Result (*transfer)(vector<FixedBits*>&, F
       current /= 3;
       }
 
-    bool first = true;
+    bool max_conflict = true;
     for (int j = 0; j < list.size(); j++)
       {
       D& d = list[j];
       if (children[0]->unsignedHolds(d.a) && children[1]->unsignedHolds(d.b) && output.unsignedHolds(d.c))
         {
-        if (first)
+        if (max_conflict)
           {
           c_a.fromUnsigned(d.a);
           c_b.fromUnsigned(d.b);
           c_o.fromUnsigned(d.c);
-          first = false;
+          max_conflict = false;
           }
         else
           {
@@ -409,14 +420,56 @@ exhaustively_check(const int bitwidth, Result (*transfer)(vector<FixedBits*>&, F
         }
       }
 
+    const int initialFixed = children[0]->countFixed() + children[1]->countFixed() + output.countFixed();
+    const int maxFixed = c_a.countFixed() + c_b.countFixed() + c_o.countFixed();
+
+
+    BBP.toAssumptions(*children[0], *children[1], output);
+    bool bb_conflict = !BBP.unitPropagate();
+    const int BBFixed = BBP.fixedCount();
+
+    bool transfer_conflict = (transfer(children, output) == CONFLICT) ;
+    const int transferFixed = children[0]->countFixed() + children[1]->countFixed() + output.countFixed();
+
+
+    if (transfer_conflict && !max_conflict)
+      FatalError("!!");
+    if (bb_conflict && !max_conflict)
+      FatalError("~~");
+
+    if (!max_conflict &&  !bb_conflict)
+    {
+        assert( maxFixed >= BBFixed);
+        assert( initialFixed <= BBFixed);
+    }
+
+    //cerr << "----";
+   // cerr << *children[0] << *children[1] << output << endl;
+  //  cerr << c_a << c_b << c_o << endl;
+
+    if (!max_conflict && !transfer_conflict)
+      {
+        assert( maxFixed >= transferFixed);
+        assert( initialFixed <= transferFixed);
+      }
+
+    assert( initialFixed <= maxFixed);
 
-    Result r = transfer(children, output);
-    if (r == CONFLICT)
+    if (max_conflict && !transfer_conflict)
+      transferBad++;
+    else if (max_conflict && !bb_conflict)
+        BBBad++;
+    else if (max_conflict)
       {
-      if(!first)
-        FatalError("Not First");
+
       }
-    else
+    else if (transferFixed != maxFixed)
+      transferBad++;
+    else if (BBFixed != maxFixed)
+        BBBad++;
+
+
+    if (!transfer_conflict && (k != BVMULT) && (k != BVDIV) && (k != BVMOD) )
       {
       if(!FixedBits::equals(*children[0],c_a))
         FatalError("First");
@@ -428,42 +481,13 @@ exhaustively_check(const int bitwidth, Result (*transfer)(vector<FixedBits*>&, F
     }
   for (int i=0; i < children.size();i++)
     delete children[i];
-}
 
-int plusF(int a, int b)
-{
-  return a+b;
-}
-
-int subF(int a, int b)
-{
-  return a-b;
-}
-
-int leftSF(int a, int b)
-{
-  if (b >= sizeof(int)*8)
-    return 0;
+  cerr.setf(ios::fixed);
+  cerr << setprecision(1);
+  cerr  << "& " <<     100 *(transferBad/ (float)to_iterate) << "\\%";
+  cerr  << "& " <<     100 *(BBBad/ (float)to_iterate) << "\\%";
+  transferBad= 0; BBBad = 0;
 
-  return a<<b;
-}
-
-int rightSF(int a, int b)
-{
-  if (b >= sizeof(int)*8)
-    return 0;
-
-  return a>>b;
-}
-
-int bvandF(int a, int b)
-{
-  return a &b;
-}
-
-int bvorF(int a, int b)
-{
-  return a | b;
 }
 
 void
@@ -507,11 +531,9 @@ main(void)
   mgr = &stp;
   mgr->UserFlags.disableSimplifications();
   mgr->UserFlags.division_by_zero_returns_one_flag  = true;
-
-  Cpp_interface interface(*mgr, mgr->defaultNodeFactory);
-  interface.startup();
+  Cpp_interface interface(*mgr);
   srand(time(NULL));
-  BEEV::ParserBM = mgr;
+
 
   // Add had a defect effecting bithWidth > 90.
   // Shifting had a defect effecting bitWidth > 64.
@@ -520,7 +542,7 @@ main(void)
 
   const int bits = 5;
 
-  if (true)
+  if (false)
     {
     output << "signed greater than equals" << endl;
     go(&bvSignedGreaterThanEqualsBothWays, BVSGE);
@@ -554,7 +576,6 @@ main(void)
 
       output << "multiplication" << endl;
       go(&multiply, BVMULT);
-
       output << "unsigned division" << endl;
       go(&unsignedDivide, BVDIV);
 
@@ -570,17 +591,50 @@ main(void)
       exit(1);
     }
 
-  const int exhaustive_bits = 6;
+
+  const int exhaustive_bits = 5;
+
+  Functions f;
+  cerr << "% Automatically generated!!" << endl;
+  cerr << "\\begin{figure} \\centering" <<endl;
+  cerr << "\\begin{tabular}{|c|";
+
+  for (int i = 1; i <= exhaustive_bits; i++)
+    cerr << "c|c|";
+
+  cerr << "} \\hline" << endl;
+
+  cerr << "Operation";
+
+  for (int i = 1; i <= exhaustive_bits; i++)
+    cerr << "& \\multicolumn{2}{c|}{"<< i <<" bits}";
+  cerr << "\\\\ \\hline" << endl;
+
+
+
   for (int i = 1; i <= exhaustive_bits; i++)
+    cerr << "& Prop& BB";
+
+  cerr << "\\\\ \\hline" << endl;
+
+  std::list<Functions::Function>::iterator it = f.l.begin();
+  while (it != f.l.end())
     {
-    exhaustively_check(i, &bvLeftShiftBothWays, &leftSF);
-    exhaustively_check(i, &bvRightShiftBothWays, &rightSF);
-    exhaustively_check(i, &bvAddBothWays, &plusF);
-    exhaustively_check(i, &bvSubtractBothWays, &subF);
-    exhaustively_check(i, &bvAndBothWays, &bvandF);
-    exhaustively_check(i, &bvOrBothWays, &bvorF);
+      Functions::Function& f = *it;
+      cerr << f.name << endl;
+      for (int i = 1; i <= exhaustive_bits; i++)
+        exhaustively_check(i, f.k,  f.fn, f.op);
+      cerr << "\\\\ " << endl;
+      it++;
     }
+cerr << "\\hline" ;
+
+  cerr << "\\end{tabular}" <<endl;
+  cerr <<  "\\caption{Percentage of all the assignments at different bit-widths. Where the Bitblasted encoding and the propagators did missed bits to fix, or missed a conflicting assignment.}" << endl;
+  cerr << "\\end{figure}" <<endl;
+
 
+exit(1);
 
 
   if (true)