Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add a tcSmtFile REPL option #1783

Merged
merged 3 commits into from
Jan 23, 2025
Merged

Add a tcSmtFile REPL option #1783

merged 3 commits into from
Jan 23, 2025

Conversation

RyanGlScott
Copy link
Contributor

@RyanGlScott RyanGlScott commented Dec 9, 2024

tcSmtFile is to tcSolver as smtFile is to prover. The tcSmtFile option allows users to record typechecker-related SMT solver interactions to a file.

This requires using a more recent simple-smt version to bring in the changes from yav/simple-smt#25 and yav/simple-smt#27, which are needed to plumb the relevant information down to the simple-smt API.

Fixes #1782.

@RyanGlScott RyanGlScott marked this pull request as ready for review December 13, 2024 20:09
@RyanGlScott RyanGlScott marked this pull request as draft December 13, 2024 20:10
@RyanGlScott RyanGlScott marked this pull request as ready for review December 13, 2024 23:28
RyanGlScott and others added 2 commits January 23, 2025 09:59
`tcSmtFile` is to `tcSolver` as `smtFile` is to `prover`. The `tcSmtFile`
option allows users to record typechecker-related SMT solver interactions to a
file.

This requires using a more recent `simple-smt` version to bring in the changes
from yav/simple-smt#25 and
yav/simple-smt#27, which are needed to plumb the
relevant information down to the `simple-smt` API.

Fixes #1782.
@RyanGlScott RyanGlScott merged commit 323e324 into master Jan 23, 2025
50 checks passed
@RyanGlScott RyanGlScott deleted the T1782-tcSmtFile branch January 23, 2025 16:28
RyanGlScott added a commit to GaloisInc/saw-script that referenced this pull request Jan 23, 2025
…Inc/cryptol#1783, and GaloisInc/cryptol#1791

This is a collection of `cryptol` submodule bumps that is squashed together in
order to ensure that the SAW CI continues to pass with this commit:

* GaloisInc/cryptol#1783 bumps Cryptol's lower version bounds on `simple-smt`
  to `>=0.9.8`, which requires updating SAW's `cabal.GHC-*.config` files
  accordingly.
* GaloisInc/cryptol#1526 is brought along for the ride as part of this bump,
  but this will not pass SAW's integration tests without also including the
  bugfix from GaloisInc/cryptol#1791. As a result, I've also included
  GaloisInc/cryptol#1791.

GaloisInc/cryptol#1791 is latest of the three patchsets, so I have bumped the
`cryptol` submodule to point to the merge commit from that PR.
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.

Offer a tcSmtFile option for writing typechecker SMT solver output to a file
2 participants