Make order of theories explicit in the source code. (#3379)
authorAina Niemetz <aina.niemetz@gmail.com>
Fri, 11 Oct 2019 01:52:46 +0000 (18:52 -0700)
committerGitHub <noreply@github.com>
Fri, 11 Oct 2019 01:52:46 +0000 (18:52 -0700)
commit91acac585b0b2bc5a3fab4466d887cfbafa35f77
tree0cb007ff0fb13c6d80675675e6375a9b5bab1be0
parent9c39dca48e7f0bff035cfff233c9d5224d9a0974
Make order of theories explicit in the source code. (#3379)

Fixes #2517.

This makes the order of theories explicit in the source code rather than relying on the order defined via the build system. Previously, the build system ensured the order of the theories via the KINDS_FILES variable, which is a list of kinds files that is fed to code generation scripts (mkkind, mkmetakind, mkrewriter, mktheorytraits). The generated code critical to the order of theories w.r.t. soundess is the TheoryId enum, and the CVC4_FOR_EACH_THEORY macro. Ideally, we would want to get rid of the latter (ugly and error prone), which is not possible in the current configuration, and to be discussed in the future.

This PR moves the TheoryID enum and related functions to theory/theory_id.h, and the CVC4_FOR_EACH_THEORY macro to theory/theory_engine.cpp, the only place where it is used.

I ran it on whole SMT-LIB (non-incremental and incremental) and did not encounter any soundness issues. The only issue that did occur is not related to these changes, non-critical and known: #2993
14 files changed:
src/CMakeLists.txt
src/cvc4.i
src/expr/expr_template.h
src/expr/kind_template.cpp
src/expr/kind_template.h
src/expr/mkkind
src/parser/smt2/smt2.cpp
src/theory/mktheorytraits
src/theory/theory.h
src/theory/theory_engine.cpp
src/theory/theory_id.cpp [new file with mode: 0644]
src/theory/theory_id.h [new file with mode: 0644]
src/theory/theory_id.i [new file with mode: 0644]
src/theory/theory_traits_template.h