Fix bdd.as_expr()

romainbrenguier · romainbrenguier · commit 8f7df386e7e2 · 2019-02-25T16:16:55.000Z
There was a mistake in the way complement is computed when going to the
then_branch.
diff --git a/src/solvers/prop/bdd_expr.cpp b/src/solvers/prop/bdd_expr.cpp
@@ -119,11 +119,11 @@ exprt bdd_exprt::as_expr(const bdd_nodet &r, bool complement) const
       if(r.else_branch().is_complement() != complement)
       {
         return and_exprt(
-          n_expr, as_expr(r.then_branch(), complement != r.is_complement()));
+          n_expr, as_expr(r.then_branch(), complement != r.then_branch().is_complement()));
       }
       return or_exprt(
         not_exprt(n_expr),
-        as_expr(r.then_branch(), complement != r.is_complement()));
+        as_expr(r.then_branch(), complement != r.then_branch().is_complement()));
     }
   }
   else if(r.then_branch().is_constant())

Original file line number	Diff line number	Diff line change
`@@ -119,11 +119,11 @@ exprt bdd_exprt::as_expr(const bdd_nodet &r, bool complement) const`
`119`	`119`	`if(r.else_branch().is_complement() != complement)`
`120`	`120`	`{`
`121`	`121`	`return and_exprt(`
`122`		`- n_expr, as_expr(r.then_branch(), complement != r.is_complement()));`
	`122`	`+ n_expr, as_expr(r.then_branch(), complement != r.then_branch().is_complement()));`
`123`	`123`	`}`
`124`	`124`	`return or_exprt(`
`125`	`125`	`not_exprt(n_expr),`
`126`		`- as_expr(r.then_branch(), complement != r.is_complement()));`
	`126`	`+ as_expr(r.then_branch(), complement != r.then_branch().is_complement()));`
`127`	`127`	`}`
`128`	`128`	`}`
`129`	`129`	`else if(r.then_branch().is_constant())`