Disable shared selectors for quantified logics without SyGuS (#7156)
authorAndrew Reynolds <andrew.j.reynolds@gmail.com>
Thu, 9 Sep 2021 01:50:09 +0000 (20:50 -0500)
committerGitHub <noreply@github.com>
Thu, 9 Sep 2021 01:50:09 +0000 (01:50 +0000)
commit76459c48a76eb0deb53377c634295b4aa5613605
tree55ebdb52816790cf9a5ce28d230eb91f270c46e8
parentdfd135ee8039c901e535b0781ae1b27cb3365166
Disable shared selectors for quantified logics without SyGuS (#7156)

The use of shared selectors may have fairly negative effects when combined with E-matching. The issue is that it allows too many things to match. When shared selectors are disabled, a selector trigger like s(x) will only match terms that access the field of the constructor associated with s. When we use shared selectors, s(x) is converted to e.g. shared_selector_D_Int_1(x), which in turn matches applications of selectors of the same type over any constructor.

This PR disables shared selectors when quantifiers are present and SyGuS is not used.
It also disables shared selectors in single invocation subcalls, thus fixes #3109.
It further makes a minor fix to the datatypes rewriter to ensure that rewritten form does not depend on options.
src/smt/set_defaults.cpp
src/theory/datatypes/datatypes_rewriter.cpp
src/theory/datatypes/datatypes_rewriter.h
src/theory/quantifiers/sygus/ce_guided_single_inv.cpp
test/regress/CMakeLists.txt
test/regress/regress1/sygus/issue3109-share-sel.sy [new file with mode: 0644]