Better normalization of string concatenation (#1719)

Documentation and simplifications for PBE (#1677)

Regression level 0 for distcheck on Travis (#1714)

We are running all our Travis tests with regression level 0 except for
distcheck, which is running with the default level 1. This commit lowers
the level to 0 to make all jobs consistent.

Add support for filtering regressions with regex (#1711)

This commit adds support for filtering the regression tests using a
regex. For example:

```
TEST_REGEX=quantifiers make regress0
```

Runs regression tests from level 0 that have "quantifiers" in their
name.

Fix memory leak in bvminisat (#1710)

While reviewing #1695, I realized that bvminisat is leaking memory for
each call to setNotify(). This commit uses std::unique_ptr to fix the
issue. It also adds std::unique_ptr to manage d_minisat.

Make Java bindings work with newer build envs (#1709)

Our current build scripts did not work with Automake 1.16. At configure
time, the .swig_deps target in src/bindings/Makefile.am was executed due
to the `@mk_include@ .swig_deps` (which is not the case with older
versions of Automake). This ultimately caused configure to fail because
SWIG was complaining about missing files (generated source files, such
as src/expr/expr.h). This commit fixes the issue by adding
`-ignoremissing` to the call to SWIG. With that option, SWIG is not
complaining about the missing files and the dependency generation
completes successfully.

Currently, the src/bindings/compat/java/create_impl.py script is not
compatible with Python 3, which leads to errors when building on systems
where `python` links to Python 3 (e.g. on Arch Linux). This commit makes
the script compatible with both Python 2 and 3.

Our build scripts were using old -source/-target versions when calling
`javac`. Those are not supported by newer Java versions (e.g.  Java 9).
This commit updates the version to 1.6, which is still fairly old, so
should be broadly supported.

Finally, some systems (e.g. Arch Linux' AUR package for SWIG 2) refer to
SWIG 2 as `swig-2`.  This commit adds support for detecting this at
configure time.

 Add reasoning for inequalities in str rewriter (#1713)

Synth-check and accelerate options for sygus-rr (#1691)

Abort when sygus-verify finds unsoundness. (#1717)

Rewrites for substr of strings of length one (#1712)

This commit adds a rewrite for substrings of strings of length one to
the empty string if it can be shown that it is not possible that the
start position and the length are both greater than zero:

```
(str.substr "A" x y) --> "" if x = 0 |= 0 >= y
```

The commit introduces a set of functions to check such entailments
with assumptions.

Check model only when sat (#1694)

Cleanup various exit calls (#1692)

Remove doc/libcvc4.3 from options/Makefile.am. (#1696)

This commit fixes an issue with calling make clean && make.

The final doc/libcvc4.3 is now generated during ./autogen.sh and should not
be deleted with make clean.

Add a few quantifiers regressions to improve coverage (#1702)

Remove abstract regular expression constant (#1698)

Remove unused code (#1700)

Minor reorganization for ematching (#1701)

Enable post-condition strenghtening by default for non-syntax restricted invariant synthesis (#1703)

Ignore whitespaces and moved code for contrib/get-authors.

Fix 'make regress' (#1683)

Commit b8db52f9bad5b1053810c93f0067de8423349da3 removed the option to do "make regress" (I only tested with "make regressX" and "make check"). This commit reenables "make regress".

Refactor mkoptions (#1631)

This commit refactors code generation for options. It uses a new configuration format for defining options (*.toml) and a new Python script mkoptions.py to generate the source code and option documentation.

The option behavior did not change for most of the options, except that for bool --enable-/--disable- long options enable/disable was removed. E.g. --enable-miplib-trick and --disable-miplib-trick got changed to --miplib-trick and --no-miplib-trick.

This commit fixes also an issues with set-option/get-option via the SMT2 interface. Before long options were only accessible if the name included the =ARG part.

Add bit-vector extract example. (#1681)

More rewrites for indexof (#1648)

 Move regression tests to single Makefile.am (#1658)

Until now, regression tests were split across tens of different
Makefile.am, which required a lot of code duplication and does not
really seem to be in the spirit of automake. If we want to change the
LOG_COMPILER/LOG_DRIVER for example, we have to change every single
Makefile.am, which is cumbersome (I was able to get something
semi-working by exporting those variables but it didn't seem very
clean). Additionally, it made the output of the regression tests fairly
verbose and split the output across multiple log files. Finally
it also limited parallelism when running the regression tests (this fix lowers
the time it takes to run regression level 1 from 3m to 1m45s on my
machine with 16 threads).

This commit moves all the regression tests into
test/regress/Makefile.tests and changes test/regress/Makefile.am to deal
with this new structure. Finally, it changes how the test summary in
test/Makefile.am is produced: instead of relying on the log files for
the subdirectories, it greps for the test results in the log files of
the individual tests. Not the most elegant solution but we should
probably anyway delegate that task to a Python script at some point.

Fix various regression tests (#1657)

While reorganizing the regression tests, I found three tests with a
wrong status, one that CVC4 reported unknown for and some regression
tests that had command line options set in the Makefile.am instead of
the tests themselves. This commit fixes the status of the three
regression tests (verified the status with Z3), adds command line
options to make the previously "unknown" test work, and moves the
command line options from the Makefile.am into the individual regression
tests. The latter also required some minor changes to the regression
script because it would not try to determine the expected output from
the (set-info :status ...) command if there was one directive (such as
EXPECT/COMMAND-LINE/etc.).  This was an issue with the tests that now
have a COMMAND-LINE directive.

Fix for string disequality processing (#1679)

Add support for CaDiCaL as eager BV SAT solver. (#1675)

Minor refactor datatypes sygus (#1673)

Internally remove redundant assertions and infer equalities in NonLinearExtension (#1633)

Fix datatype dump regression. (#1672)

Minor fix and addition to sygus sampler (#1678)

Add parameterized datatypes example. (#1676)

correct instruction for running example (#1669)

Enable CEGQI for non-linear (#1674)

Document inferences for strings (#1642)

Use Cryptominisat version 5.0.2 (instead of 4.2.0). (#1664)

SmtEngine::getModel() is now public. (#1665)

Printers are now managed as unique_ptr (fix mem leak). (#1654)

Some minor cleanup in bv::utils. (#1663)

Add support for SMT-LIB v2.5 command get-unsat-assumptions (#1653)

Skip (get-unsat-assumptions) tests not supported (#1656)

(get-unsat-assumptions) requires the proof infrastructure,so we can't
run the regression tests if CVC4 has not been configured for it. This
commit changes the regression script to skip tests containing
(get-unsat-assumptions) when CVC4 has not been compiled with proof
support.

Cleanup Cryptominisat SAT wrapper. (#1652)

Cryptominisat has name conflicts with the other Minisat implementations since
the Minisat implementations export var_Undef, l_True, ... as macro whereas
Cryptominisat uses static const. In order to avoid these conflicts we
forward declare CMSat::SATSolver and include the cryptominisat header only
in cryptominisat.cpp.

Further, the helper functions are moved into an anonymous namespace in the .cpp file and functions that were not used are removed.

Fixed message in get-antlr script.

Fix Travis for unit test compilation errors. (#1651)

make units does not fail if we have compile error for a unit test, however, make check does.
-Wsuggest-override is now explicitly disabled for unit tests since CxxTest does not add override keywords to the generated source code and thus get a lot of compiler warnings.

Further, this fixes some issues introduced with #1647 due to make units not failing on Travis and fixes the nightly builds.

Make statistics output consistent. (#1647)

* Fixes --hide-zero-stats (and really skips the 0 values)
* Removes the additional newline after each statistic
* Introduces theory::getStatsPrefix(TheoryId) to generate consistent
  prefixes for statistics based on the theory id
  (e.g., THEORY_BV -> "theory::bv").

Simplify initialization of quantifiers engine (#1641)

Refactor symmetry breaking in datatypes sygus (#1640)

Remove printf from output utilities (#1629)

This commit removes the unused printf functions from the output utilities. It also adds `const` keywords where possible. Finally, it removes overloaded `const char*` functions if the same function existed for `const std::string&` and the `const char*` version was only casting the `const char*` to an `std::string`. This conversion happens implicitly, so the `const char*` version is not needed.

Update semantics for string indexof and replace (#1630)

Fix boost url in contrib/get-win-dependencies.

This fixes the broken windows nightly builds.

Enable -Wsuggest-override by default. (#1643)

Adds missing override keywords.

Fix for sampler. (#1639)

Add support for check-sat-assuming. (#1637)

This adds support for check-sat-assuming. It further adds support for SmtEngine::query() over a
vector of Expressions, e.g., smtEngine->query({a, b}); checks the validity (of the current input
formula) under assumption (not (or a b)).

Add CVC4_PUBLIC keyword to overloads of << for Expr containers. (#1644)

Add uniform way to serialize containers of Expr to stream. (#1638)

Fixed typo.

Optimization for sygus streaming mode (#1636)

Simplify sygus wrt miniscoping (#1634)

Fixed comments in smt_engine.h.

Print candidate rewrites in terms of original grammar (#1635)

Create infrastructure for sygus modules (#1632)

SmtEngine::getAssignment now returns a vector of assignments. (#1628)

Minor fixes for rec-fun (#1616)

Remove unused code in pushDefineFunRecScop in smt2.cpp. (#1627)

Improve rewriter for string indexof (#1592)

Option to not use partial function semantics for arithmetic div by zero (#1620)

Improvements to sygus sampling (#1621)

Add unit tests for BitVector, minor BV rewrite fix (#1622)

This commit adds unit tests for the BitVector class and adds some additional argument checks. Additionally, it fixes a minor issue in the ZeroExtendUltConst rule if the zero_extend was by 0 bits. In that case, the rule was calling BitVector::extract() with high < low.

Split and document bitvector.h. (#1615)

Fix cd-simplification for strings (#1624)

Minor improvements to string rewriter (#1572)

Fixed disabling the BV equality slicer for quantifiers. (#1623)

This fixes an incorrect condition introduced in #1619 to disable the BV equality slicer.

Disable BV equality slicer if not pure QF_BV. (#1619)

Improve documentation of bv::utils::isCoreTerm (#1617)

Moved and simplified bv::utils::intersect. (#1614)

Tested against the recursive implementation with a temporary assertion on regression tests
with --bv-eq-slicer=auto.

Minor fixes and additions for transcendental functions (#1612)

Also adds parsing support for PI in smt2 with syntax "real.pi".

Unrecursified and merged bv::utils::is(Core|Equality)Term. (#1605)

This unrecursifies and merges bv::utils::isCoreTerm and bv::utils::isEqualityTerm to avoid
code duplication.

In the best case, the recursive implementation visits less nodes. This can be achieved with
the non-recursive implementation, however, at the cost of increased code complexity.
In practice, on QF_BV the new implementation slightly improves performance.

Tested against the recursive implementation with a temporary assertion on regression
tests (bv::utils::isCoreTerm was tested with --bv-eq-slicer=auto).
Further tested on QF_BV with a TO of 300s (slight performance improvement).

bv::utils::mk(And|Or) Do not return true if size of vector is 0. (#1613)

Make regress1 default, only test regress0 on Travis. (#1611)

Removed bv::utils::mkConjunction (redundant). (#1610)

Fix corner case for rewrite of mult by pow 2 (#1601)

Refactor regressions (#1581)

Fix context memory manager unit test (#1609)

Commit 83f150c727f197c530d6f46a75b516eea52bed29 changed the flag
that we use to determine whether to use the context memory manager
or the debug version. However, the change was not reflected in
the context memory manager unit test. This commit fixes the unit
test to check the correct flag.

Quantifiers subdirectories (#1608)

Remove unused cd_set_collection.h (#1606)

Provide a uniform way to serialize node containers to output stream. (#1604)

Moved (unrecursified) bv::utils::collectVars. (#1602)

Skip header for determining top contributors list. (#1603)

Option to use extended rewriter as a preprocessing pass (#1600)

Minor improvements to sygus sampler (#1598)

Move (unrecursified) bv::utils::numNodes to lazy_bitblaster.cpp. (#1594)

This unrecursifies and moves bv::utils::numNodes to an unnamed namespace in lazy_bitblaster.cpp
(only place where it is used). Tested against the recursive implementation (with a temporary
Assertion) on regression tests.

More minor improvements to synth-rr (#1597)

Move BitVector specific funs from bv::utils to util/bitvector.h. (#1589)

Remove mkNode from bv::utils (#1587)

Removing an always true comparison (unsigned) >= 0u.  (#1582)

Class to reduce printing of redundant candidate rewrites (#1588)

Renaming CHECK to CVC4_CHECK. This avoids name collisions with other popular assertion macros. This name is likely temporary while Assert() is deprecated. (#1590)

Replacing an incorrect reference to an injected class name when the type was meant. (#1585)

Replace CMM flag with debug CMM flag, fix leak in debug CMM (#1586)

Previously, we had -DCVC4_CONTEXT_MEMORY_MANAGER that needed to be added as a compile flag
to use the context memory manager (which we want by default). This makes compiling with
other build systems cumbersome because you have to know about the flag.

This commit replaces the -DCVC4_CONTEXT_MEMORY_MANAGER flag with a -DCVC4_DEBUG_CONTEXT_MEMORY_MANAGER
flag that does the opposite (in absence of the flag, we use the context memory manager).
Additionally, the commit fixes a memory leak in the debug context memory manager (the
destructor did not clean up the allocations).

Inlining line_buffered_input to avoid warning about unused variables in production builds. (#1584)