Add skolem definition manager (#6187)
authorAndrew Reynolds <andrew.j.reynolds@gmail.com>
Mon, 22 Mar 2021 17:30:21 +0000 (12:30 -0500)
committerGitHub <noreply@github.com>
Mon, 22 Mar 2021 17:30:21 +0000 (12:30 -0500)
commit64abc6827ec78183605db53e5dd8e2a7a0db59ed
tree42b38f99bf9ad70cc2991fb015848a4913bdbb57
parent30034769128894884a94b1b99feb95737f163bab
Add skolem definition manager (#6187)

This creates a central utility for managing "skolem definitions", e.g. mapping between skolems and the lemmas that define their behavior.

This utility is taken from the satRlv branch. It will also be used for the new implementation of the justification decision heuristic.

Note that this PR takes some helper functions out of term formula removal (e.g. hasSkolems) Prior to this PR, these helper functions were incorrect since term formula removal does not account for all introduced skolems. For instance, Theory::ppRewrite may introduce skolems directly. This PR consolidates these cases into the new class, which is called from PropEngine when lemmas and assertions are added. At the moment, the only use of this method is for CEGQI, which needs to do its own tracking of skolems in certain literals.

It also makes some minor reorganization to prop engine.
src/CMakeLists.txt
src/prop/prop_engine.cpp
src/prop/prop_engine.h
src/prop/skolem_def_manager.cpp [new file with mode: 0644]
src/prop/skolem_def_manager.h [new file with mode: 0644]
src/prop/theory_proxy.cpp
src/prop/theory_proxy.h
src/smt/term_formula_removal.cpp
src/smt/term_formula_removal.h