Skip to content

PR to track HOL changes, don't merge#822

Open
mezpusz wants to merge 221 commits into
masterfrom
ahmed-new-hol
Open

PR to track HOL changes, don't merge#822
mezpusz wants to merge 221 commits into
masterfrom
ahmed-new-hol

Conversation

@mezpusz
Copy link
Copy Markdown
Contributor

@mezpusz mezpusz commented Mar 13, 2026

No description provided.

ibnyusuf and others added 30 commits July 19, 2022 12:07
Comment thread Shell/Property.cpp
Comment thread Shell/NewCNF.cpp

GenClause::Iterator lit = gc->genLiterals();
while (lit.hasNext()) {
GenLit gl = lit.next();
Formula* g = formula(gl);

// This can happen when the problem is pseudo-higher-order
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

TODO check why

Comment thread Shell/Preprocess.cpp
Comment thread Shell/Property.hpp
// for use by polymorphic unit tests
// have to force the type con arity to avoid
// running KBOforEPR
void forceMaxTypeConArity() { _maxTypeConArity = 1; }
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No KBOforEPR anymore, so this is obsolete.

Comment thread Shell/Skolem.cpp
Comment thread SAT/Z3Interfacing.cpp
}
} else {
symb = env.signature->getFunction(trm->functor());
range_sort = SortHelper::getResultSort(trm);
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

TODO check what these do

Comment thread Shell/TheoryAxioms.cpp
if (env.options->FOOLParamodulation()) {
if (env.options->FOOLParamodulation()
#if VHOL
|| env.options->cases() || env.options->casesSimp() || env.property->higherOrder()
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The last disjunct is questionable, as by default nothing handles FOOL exhaustiveness in the HOL branch, losing some problems.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants