Simplify computation of relevant terms in datatypes (#6885)
authorAndrew Reynolds <andrew.j.reynolds@gmail.com>
Thu, 22 Jul 2021 16:57:46 +0000 (11:57 -0500)
committerGitHub <noreply@github.com>
Thu, 22 Jul 2021 16:57:46 +0000 (16:57 +0000)
commit2bf6f49039b7c9fee6a7da846e329eb9be75713c
treece2b05ffd41a5ad240fc05db6a4a47f4c18d6d54
parent88ab520f89f971a9d83d18a08a2aa5cea493a6a5
Simplify computation of relevant terms in datatypes (#6885)

This makes it so that we do not consider non-singleton equivalence classes of DT for non-datatype terms to be relevant. This impacts how model construction is done for datatypes.

It's not clear why this method considered such terms to be relevant, I believe this was done to mask a previous bug; regressions now pass when this method is simplified.

Doing this is definitely wrong when using central equality engine, hence the motivation for this simplification.
src/theory/datatypes/theory_datatypes.cpp