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

P2992 R1 Attribute [[discard("reason")]] #1665

Open
wg21bot opened this issue Oct 16, 2023 · 5 comments
Open

P2992 R1 Attribute [[discard("reason")]] #1665

wg21bot opened this issue Oct 16, 2023 · 5 comments
Labels
EWG Evolution SG22 C / C++ liaison
Milestone

Comments

@wg21bot
Copy link
Collaborator

wg21bot commented Oct 16, 2023

P2992R0 Attribute [[discard]] and attributes on expressions (Giuseppe D'Angelo)

@wg21bot wg21bot added EWG Evolution EWGI Evolution Incubator SG22 C / C++ liaison labels Oct 16, 2023
@wg21bot wg21bot added this to the 2023-telecon milestone Oct 16, 2023
@jfbastien jfbastien removed the EWG Evolution label Oct 21, 2023
@erichkeane
Copy link
Collaborator

erichkeane commented Nov 7, 2023

EWGI discussed P2992R0 during the Monday Afternoon session in Kona. The following polls were taken:

EWGI believes the application of [[discard]] on expressions is well motivated, and should be included in future versions of the paper.

SF F N A SA
0 2 4 4 0

Result: Not Consensus

Given the committee's limited time, EWGI believes P2992R0 is sufficiently developed and motivated to forward to EWG.

SF F N A SA
1 5 3 2 0

Result: Consensus

@erichkeane erichkeane added EWG Evolution and removed EWGI Evolution Incubator labels Nov 7, 2023
@wg21bot
Copy link
Collaborator Author

wg21bot commented Feb 16, 2024

P2992R1 Attribute [[discard("reason")]] (Giuseppe D'Angelo)

@wg21bot wg21bot modified the milestones: 2023-telecon, 2024-telecon Feb 16, 2024
@wg21bot wg21bot changed the title P2992 R0 Attribute [[discard]] and attributes on expressions P2992 R1 Attribute [[discard("reason")]] Feb 16, 2024
@cor3ntin cor3ntin changed the title P2992 R1 Attribute [[discard("reason")]] P2992 R1 Attribute [[discard("reason")]] Feb 16, 2024
@erichkeane
Copy link
Collaborator

Note that we shouldn't see this paper until #1761 develops further as that provides the infrastructure to allow that to happen. That said, the 'split' of the papers should likely go the 'other way'. As this is the paper that was forwarded, THIS should contain the "attribute on expressions" paper (as procedurally, that was what was forwarded), and the [[discard("reason")]] should happen separately/needs to go to EWGI.

@dangelog
Copy link

Hi,

I might have misunderstood the results of the polls. This paper's R0 proposed both the [[discard]] attribute, as well as attributes on expressions. In there, [[discard]] was proposed as an attribute on statements or expressions.

The first poll got no consensus for keeping [[discard]] on expressions. So in R1 I reduced the paper to just proposing the attribute, and propose it on statements only. This is self-contained -- it doesn't depend on the other paper, although this results in a smaller applicability.

Proposing attributes on expressions (in "general") got instead split in P3093. Of course that feature has a direct use-case in the context of the [[discard]] attribute; but, as I said, I interpreted the poll as a lack of appetite for it. Clearly, if both features get individually merged, I'm going to ask to re-evaluate that poll and make [[discard]] work on expressions as well.

Did I get the split wrong? Possibly; I'm happy to rework this if it makes things easier, or present both papers at the same time.

@jensmaurer
Copy link
Member

This paper tracker is for administrative purposes only. If there are questions about the meaning of polls and the next steps of a paper, please discuss them directly with the chair of the respective subgroup.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
EWG Evolution SG22 C / C++ liaison
Projects
None yet
Development

No branches or pull requests

5 participants