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

[type.traits] Strike 'at compile time' #6841

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

Eisenwave
Copy link
Contributor

@Eisenwave Eisenwave commented Mar 1, 2024

Related to #5641.

C++ doesn't have a notion of "compile-time" or "run-time". We can simply say that type traits query the properties of a type, period.

If we really wanted to, we could say something about constant expression here. However, I see this as pointless; these paragraphs are informative anyway.

@Eisenwave Eisenwave changed the title [meta.unary.general] Strike 'at compile time' [type.traits] Strike 'at compile time' Mar 1, 2024
@jwakely
Copy link
Member

jwakely commented Mar 1, 2024

If we wanted to retain this in some form, we could say "during translation". I don't think we lose anything by striking it completely though.

@Eisenwave
Copy link
Contributor Author

I don't think we lose anything by striking it completely though.

Me neither. The paragraphs already mention that the subclauses contain templates, not utility functions or something that would suggest run-time over compile-time.

If we wanted to retain this in some form, we could say "during translation".

If we added "during translation", that may beg the question "during translation of what?" Would that justify an \iref{lex.phases}? Honestly it's not worth the trouble to replace the wording with an equivalent.

@jwakely
Copy link
Member

jwakely commented Mar 2, 2024

During translation of the program. That's how we refer to compilation, so something that happens at compile time happens during translation. I don't think anything more would be needed.

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.

None yet

2 participants