Skip to content

Fix #613 MCSAT curried function model reconstruction#614

Open
disteph wants to merge 4 commits intomasterfrom
issue-613-mcsat-curried-function-model
Open

Fix #613 MCSAT curried function model reconstruction#614
disteph wants to merge 4 commits intomasterfrom
issue-613-mcsat-curried-function-model

Conversation

@disteph
Copy link
Copy Markdown
Collaborator

@disteph disteph commented Mar 11, 2026

Fixes #613.

This fixes an MCSAT model-construction bug on curried functions such as array(1)(7), which could reach sat and then crash while building/printing the model. The UF model builder now reconstructs nested function values recursively from the MCSAT trail instead of treating function-valued intermediates as first-order defaults.

This PR also adds a focused API regression test for the reported curried-function case, fixes an MCSAT-specific lambda error-path mismatch in the SMT2 preprocessor, and extends the regression harness with a tests/regress/both bucket that runs selected higher-order/array regressions under both --mcsat and --dpllt.

@coveralls
Copy link
Copy Markdown

Coverage Status

coverage: 66.68% (+0.1%) from 66.539%
when pulling dc5687c on issue-613-mcsat-curried-function-model
into 016a624 on master.

@ahmed-irfan ahmed-irfan self-requested a review March 13, 2026 05:38
daniel-raffler added a commit to sosy-lab/java-smt that referenced this pull request Mar 13, 2026
Segfaults have been fixed by SRI-CSL/yices2#614
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

MCSAT crashes when generating models for curried functions

2 participants