Minor fixes of policy for eliminating quantifiers (#7033)
authorAndres Noetzli <andres.noetzli@gmail.com>
Wed, 18 Aug 2021 21:06:52 +0000 (14:06 -0700)
committerGitHub <noreply@github.com>
Wed, 18 Aug 2021 21:06:52 +0000 (16:06 -0500)
commit7ef9f24a1a879d278d8ac5a3fef28c7ca8489d2c
tree57b1dee485832aece072146a378ecccabc91d83e
parentdfd844226c4cc6c6d2af78ef6291b0a9d087f4d4
Minor fixes of policy for eliminating quantifiers (#7033)

PR #7017 fixed the policy for eliminating quantifiers but introduced
some minor issues, which this commit fixes:

the InstantiationEngine::checkOwnership() method was changed to look
for patterns in the wrong node.
the PR changed the modes of the --user-pat=MODE method but reused
one of the names. This commit fixes the name and adds a check in the
options script.
fixing the policy caused cvc5 to answer unsat instead of the
expected unknown for regress0/use_approx/bug812_approx.smt2. The
commit updates the expected result.
src/options/mkoptions.py
src/options/quantifiers_options.toml
src/theory/quantifiers/ematching/instantiation_engine.cpp
test/regress/regress0/use_approx/bug812_approx.smt2