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

chore: use built-in SAT solver for CBMC proofs #781

Merged
merged 1 commit into from
Apr 25, 2024

Conversation

tautschnig
Copy link
Contributor

Description of changes:

Invoking an external solver is not necessarily cheaper in terms of CPU or memory usage.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Check any applicable:

  • n/a Were any files moved? Moving files changes their URL, which breaks all hyperlinks to the files.

@tautschnig tautschnig force-pushed the cbmc-use-builtin-solver branch 4 times, most recently from 1391595 to fdbb993 Compare April 25, 2024 07:24
@tautschnig
Copy link
Contributor Author

With #780 and this one we have now reduced the time for running proofs from around 1 h 15 min down to 40 mins.

@tautschnig tautschnig marked this pull request as ready for review April 25, 2024 08:03
@tautschnig tautschnig requested a review from a team as a code owner April 25, 2024 08:03
ajewellamz
ajewellamz previously approved these changes Apr 25, 2024
@tautschnig tautschnig temporarily deployed to continuous-integration April 25, 2024 12:11 — with GitHub Actions Inactive
@tautschnig tautschnig temporarily deployed to continuous-integration April 25, 2024 12:11 — with GitHub Actions Inactive
Invoking an external solver is not necessarily cheaper in terms of CPU
or memory usage.
@tautschnig tautschnig temporarily deployed to continuous-integration April 25, 2024 15:29 — with GitHub Actions Inactive
@tautschnig tautschnig deployed to continuous-integration April 25, 2024 15:29 — with GitHub Actions Active
@ajewellamz ajewellamz merged commit 42f7af6 into aws:master Apr 25, 2024
7 checks passed
@tautschnig tautschnig deleted the cbmc-use-builtin-solver branch April 26, 2024 07:43
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.

2 participants