Skip to content

Support dyn Trait with --monomorphize#1042

Merged
Nadrieril merged 85 commits intoAeneasVerif:mainfrom
Sam-Ni:dyn-mono-normal
Mar 24, 2026
Merged

Support dyn Trait with --monomorphize#1042
Nadrieril merged 85 commits intoAeneasVerif:mainfrom
Sam-Ni:dyn-mono-normal

Conversation

@Sam-Ni
Copy link
Copy Markdown
Contributor

@Sam-Ni Sam-Ni commented Mar 1, 2026

This is a draft PR implementing dynamic traits in mono mode. The implementation follows the proposal described in #856.

Currently, user-defined traits with generics and associated types are supported.

Some special cases still need to be handled to pass all test cases, including built-in traits (e.g., closures) and traits with default methods.

@Sam-Ni
Copy link
Copy Markdown
Contributor Author

Sam-Ni commented Mar 8, 2026

All comments are addressed in the latest commits.

@Sam-Ni
Copy link
Copy Markdown
Contributor Author

Sam-Ni commented Mar 9, 2026

All tests are passed in this commit.

@Sam-Ni
Copy link
Copy Markdown
Contributor Author

Sam-Ni commented Mar 10, 2026

Charon version is updated to pass check-version-number.
I think this PR is ready for review.

@Sam-Ni Sam-Ni marked this pull request as ready for review March 10, 2026 02:46
@Nadrieril Nadrieril self-assigned this Mar 16, 2026
Copy link
Copy Markdown
Member

@Nadrieril Nadrieril left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Alright, I had a good look, looks good to me! Thanks for this work!

@Nadrieril Nadrieril added this pull request to the merge queue Mar 24, 2026
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Mar 24, 2026
@Nadrieril
Copy link
Copy Markdown
Member

Seems like there was an incompatible change on main, sorry for taking so long. You'll have to rebase/merge main.

@Sam-Ni
Copy link
Copy Markdown
Contributor Author

Sam-Ni commented Mar 24, 2026

Seems like there was an incompatible change on main, sorry for taking so long. You'll have to rebase/merge main.

It' fine. Now this commit has updated to the latest main branch.

@Nadrieril Nadrieril enabled auto-merge March 24, 2026 13:58
@Nadrieril Nadrieril added this pull request to the merge queue Mar 24, 2026
Merged via the queue into AeneasVerif:main with commit c84b66a Mar 24, 2026
5 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.

2 participants