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

Big cleanup #45

Merged
merged 42 commits into from
Nov 22, 2023
Merged

Big cleanup #45

merged 42 commits into from
Nov 22, 2023

Conversation

sonmarcho
Copy link
Member

@sonmarcho sonmarcho commented Nov 22, 2023

This is the companion PR of AeneasVerif/charon#50.
In addition to propagating changes coming from Charon and using the new functionalities provided by Charon (for instance, the name matchers to map external definitions to definitions in the standard library, and the computation of the regions hierarchy) we performed the following modifications:

  • cleaned up the "Extract" files, to remove the formatter type, which proved not useful anymore and quite cumbersome.
  • add code spans in the comments of the generated code
  • use typing information to generate names for the trait clauses and the trait parent clauses. For instance, if a function requires a trait clause for Trait<T, u32>, we now generate a name like TraitTU32Inst, instead of inst followed by the index of the clause.

@sonmarcho sonmarcho merged commit bacf3f5 into main Nov 22, 2023
4 checks passed
@sonmarcho sonmarcho deleted the son_merge_types branch November 22, 2023 14:06
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.

1 participant