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

fix: Invalid declaration errors with /typeEncoding:m #4275

Merged
merged 7 commits into from
Jul 13, 2023

Conversation

zafer-esen
Copy link
Collaborator

Attempting verification directly from Dafny using /typeEncoding:m sometimes leads to invalid declaration errors in existing integration tests. The issue does not happen when printing to a Boogie file first, then verifying that file directly with Boogie. This PR tries to fix this.

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

zafer-esen and others added 5 commits July 11, 2023 16:36
Attempting verification directly from Dafny using /typeEncoding:m
sometimes leads to invalid declaration errors. The issue does not
happen when printing to a Boogie file and then verifying that file
directly with Boogie.
)

Multiple class filters must be passed in as a single
argument. See:

https://github.com/danielpalme/ReportGenerator#usage--command-line-parameters),

but previously they were passed as multiple `-classfilters:` of which
only the first was actually applied.

By submitting this pull request, I confirm that my contribution
is made under the terms of the MIT license.
- Reorder output in dafny0/Fuel.dfy.expect.
- Remove a post-condition in dafny0/FunctionSpecifications.dfy
  and update its .expect file.
  (This post-condition started timing out after the change.)
- DivergentPost was sometimes timing out during verification,
  it is now split into two: GoodPost and DivergentPost.
- Removed deprecation flag and deprecated semicolons.
@atomb atomb enabled auto-merge (squash) July 13, 2023 00:41
@atomb atomb merged commit ce78bf0 into dafny-lang:master Jul 13, 2023
19 checks passed
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.

3 participants