Skip to content

Skip C++ types when generating Destroy witnesses #6732

Merged
jonmeow merged 2 commits intocarbon-language:trunkfrom
jonmeow:destroy-skip-cpp
Feb 14, 2026
Merged

Skip C++ types when generating Destroy witnesses #6732
jonmeow merged 2 commits intocarbon-language:trunkfrom
jonmeow:destroy-skip-cpp

Conversation

@jonmeow
Copy link
Contributor

@jonmeow jonmeow commented Feb 13, 2026

This TODO had been written before C++ types were generating destroy implementations, which is resolved now.

Assisted-by: Google Antigravity with Gemini 3 Flash

@jonmeow jonmeow requested a review from a team as a code owner February 13, 2026 00:25
@jonmeow jonmeow requested review from chandlerc and removed request for a team February 13, 2026 00:25
@jonmeow
Copy link
Contributor Author

jonmeow commented Feb 13, 2026

Depends on #6731, review should focus on 500f2b5

Copy link
Contributor

@chandlerc chandlerc left a comment

Choose a reason for hiding this comment

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

Generally makes sense, but a question about enums inline.

Copy link
Contributor

@chandlerc chandlerc left a comment

Choose a reason for hiding this comment

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

LG!

Assisted-by: Google Antigravity with Gemini 3 Flash
Assisted-by: Google Antigravity with Gemini 3 Flash
@jonmeow jonmeow added this pull request to the merge queue Feb 14, 2026
Merged via the queue into carbon-language:trunk with commit 64e3fab Feb 14, 2026
8 checks passed
@jonmeow jonmeow deleted the destroy-skip-cpp branch February 14, 2026 01:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants