Skip to content

P3508R0 and P3369R0 constexpr for specialized memory algorithms #7465

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 2 commits into from
Dec 17, 2024

Conversation

jensmaurer
Copy link
Member

Fixes #7426
Also fixes cplusplus/papers#2026
Also fixes cplusplus/papers#2141

@jensmaurer jensmaurer added this to the post-2024-11 milestone Nov 30, 2024
@cpplearner
Copy link
Contributor

Should this bump __cpp_lib_constexpr_memory, following the policy documented in https://wg21.link/sd6#constexpr?

@jensmaurer
Copy link
Member Author

jensmaurer commented Dec 15, 2024

__cpp_lib_raw_memory_algorithms is being bumped. I agree that's odd given SD-6 and the presence of __cpp_lib_constexpr_memory.

@jwakely, what's going on here?

@jwakely
Copy link
Member

jwakely commented Dec 15, 2024

Inconsistent application/recollection of the policy

@jensmaurer
Copy link
Member Author

Anything we want to do here editorially? My inclination would be "no", right now.

@tkoeppe tkoeppe force-pushed the motions-2024-11-lwg-14 branch from 91ba0b2 to 3cf550d Compare December 17, 2024 12:47
@jwakely
Copy link
Member

jwakely commented Dec 17, 2024

Anything we want to do here editorially? My inclination would be "no", right now.

I agree. We can fix it as an LWG issue if needed

@tkoeppe tkoeppe force-pushed the motions-2024-11-lwg-14 branch from 3cf550d to f5b7bb8 Compare December 17, 2024 13:09
@tkoeppe
Copy link
Contributor

tkoeppe commented Dec 17, 2024

I'll merge as-is, but we can follow-up if the FT macro needs fixing.

@tkoeppe tkoeppe merged commit e887fd8 into main Dec 17, 2024
4 checks passed
@jensmaurer jensmaurer deleted the motions-2024-11-lwg-14 branch February 15, 2025 22:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
4 participants