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.

Sign up for GitHub

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

Jump to bottom

Fix bug where the ctor for structs with VTable was not enforced #520

Merged
merged 1 commit into from Apr 21, 2024
Merged

Fix bug where the ctor for structs with VTable was not enforced #520

merged 1 commit into from Apr 21, 2024

Conversation

marcauberer
Copy link
Member

  • Fix bug where the ctor for structs with VTable was not enforced
  • Fix bug where the default ctor/copy ctor was not generated although the struct has a VTable, that needs to be initialized
  • Fix bug where default ctor/copy ctor/dtor was not generated in the correct order, which is declaration order
  • Add tests

@marcauberer marcauberer added the bug Something isn't working label Apr 21, 2024
@marcauberer marcauberer added this to the 0.20.0 milestone Apr 21, 2024
@marcauberer marcauberer requested a review from a team as a code owner April 21, 2024 02:56
@marcauberer marcauberer enabled auto-merge (squash) April 21, 2024 02:56
@github-actions github-actions bot added bootstrap Change regarding the bootstrap compiler labels Apr 21, 2024
@marcauberer marcauberer merged commit 230e6e7 into main Apr 21, 2024
5 checks passed
@marcauberer marcauberer deleted the fix/vtable-requires-ctor branch April 21, 2024 03:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
bootstrap Change regarding the bootstrap bug Something isn't working compiler size/L
Projects
None yet
Milestone
0.20.0
Development

Successfully merging this pull request may close these issues.

None yet
1 participant
@marcauberer