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
[allocator.requirements.general] Dismantle requirements table #5327
Conversation
@tkoeppe, here you are. You can cherry-pick only the first commit if you don't want the whole thing. |
Wonderful, thanks! @jwakely, @CaseyCarter: would you perhaps be able to take a look at this requirements table replacement? Do you think it needs LWG oversight? |
@JohelEGP , thanks for the review. All trivial fixes are applied. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't think this needs review/approval from LWG. There is unanimous support for these changes among LWG telecon attendees.
If some minor problems do get introduced by this change, they can be fixed later. We don't need to hold the change back waiting for LWG review.
Thanks, @jwakely! Merging; please get in touch if there are any complaints or issues. |
Fixed #5310