Skip to content

[lex.separate] No more instantiation units #8069

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

Merged
merged 1 commit into from
Jul 17, 2025

Conversation

AlisdairM
Copy link
Contributor

The term "instantiation unit" was removed by the application of P2996 for reflection, #7930. This PR removes the last remaining reference from a note.

@AlisdairM AlisdairM changed the title No more instantiation units [lex.separate] No more instantiation units Jul 15, 2025
calls to functions whose identifiers have external or module linkage,
manipulation of objects whose identifiers have external or module linkage, or
Copy link
Contributor

Choose a reason for hiding this comment

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

Yes, I realize semantics linebreaks would be better, but can we keep this edit to a minimum? Just the first part is fine.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Simplified down to single commit touching only the affected lines, rebased, and force-pushed.

@AlisdairM AlisdairM force-pushed the no_more_instantiation_units branch from e05fd1d to 2daeed2 Compare July 17, 2025 14:06
The term instantiation unit was removed by the reflection paper, P2996.
@AlisdairM AlisdairM force-pushed the no_more_instantiation_units branch from 2daeed2 to 36bb3e7 Compare July 17, 2025 14:08
@tkoeppe tkoeppe merged commit 1acddce into cplusplus:main Jul 17, 2025
2 checks passed
@AlisdairM AlisdairM deleted the no_more_instantiation_units branch July 17, 2025 15:05
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