Remove dead code in bv-to-bool preprocessing pass (#1828)

Fixes Coverity issue 1468436. As Coverity correctly detects,
kind::BITVECTOR_XOR is dealt with in an if-statement before the switch
statement on kind. This is because kind::XOR is binary while
kind::BITVECTOR_XOR is n-ary (as a comment in the code correctly
indicates).

diff --git a/src/preprocessing/passes/bv_to_bool.cpp b/src/preprocessing/passes/bv_to_bool.cpp
index b01a60031f6e8082184d6d91e951b43b7fcc3c59..745a16979cc027b7e1914c333789a4741b965bd5 100644 (file)
--- a/src/preprocessing/passes/bv_to_bool.cpp
+++ b/src/preprocessing/passes/bv_to_bool.cpp
@@ -214,7 +214,6 @@ Node BVToBool::convertBvTerm(TNode node)
   {
     case kind::BITVECTOR_OR: new_kind = kind::OR; break;
     case kind::BITVECTOR_AND: new_kind = kind::AND; break;
-    case kind::BITVECTOR_XOR: new_kind = kind::XOR; break;
     case kind::BITVECTOR_NOT: new_kind = kind::NOT; break;
     default: Unhandled();
   }