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

P1938 if consteval #677

Closed
wg21bot opened this issue Oct 15, 2019 · 13 comments
Closed

P1938 if consteval #677

wg21bot opened this issue Oct 15, 2019 · 13 comments
Labels
C++23 Targeted at C++23 CWG Core LWG Library plenary-approved Papers approved for inclusion in their target vehicle by plenary vote.
Milestone

Comments

@wg21bot
Copy link
Collaborator

wg21bot commented Oct 15, 2019

P1938R0 if consteval (Barry Revzin, Daveed Vandevoorde, Richard Smith)

@wg21bot wg21bot added the EWG Evolution label Oct 15, 2019
@wg21bot wg21bot added this to the 2019-11 milestone Oct 15, 2019
@jensmaurer jensmaurer added the C++20 Targeted at C++20 label Oct 15, 2019
@jfbastien jfbastien added this to C++20 in EWGBelfast2019 Oct 15, 2019
@jfbastien jfbastien removed the C++20 Targeted at C++20 label Nov 7, 2019
@jensmaurer jensmaurer modified the milestones: 2019-11, 2020-02 Dec 11, 2019
@jfbastien jfbastien added this to Unscheduled in EWG Prague Jan 22, 2020
@jfbastien jfbastien moved this from Unscheduled to Thursday in EWG Prague Jan 23, 2020
@jfbastien jfbastien moved this from Thursday to Friday in EWG Prague Feb 12, 2020
@jfbastien
Copy link
Collaborator

See the polls from the Belfast discussion of FR222:
cplusplus/nbballot#219 (comment)

@jfbastien
Copy link
Collaborator

jfbastien commented Feb 14, 2020

EWG Prague on Friday morning:

Reviewing R1 of the paper, which wasn’t in the mailing. Can’t poll for tentative ready in this meeting.

We are interested in some solution in the space which this paper tackles, either as proposed in the paper or in some other form.

SF F N A SA
21 19 0 0 0

Consensus.

We want to pursue this paper for C++23.

SF F N A SA
10 6 15 2 0

Given how many neutrals we have here, and none in the first poll, this isn't consensus. People interested in other options should come with new information by next meeting to help increase consensus.

@jfbastien jfbastien added the needs-revision Paper needs changes before it can proceed label Feb 14, 2020
@jensmaurer jensmaurer removed this from the 2020-02 milestone Feb 18, 2020
@wg21bot
Copy link
Collaborator Author

wg21bot commented Mar 24, 2020

P1938R1 if consteval (Barry Revzin, Daveed Vandevoorde, Richard Smith, Andrew Sutton)

@wg21bot wg21bot added this to the 2020-telecon milestone Mar 24, 2020
@jfbastien jfbastien removed the needs-revision Paper needs changes before it can proceed label Mar 24, 2020
@jfbastien
Copy link
Collaborator

Discussed in today's EWG telecon. Polls:

if consteval should be parenthesized as if (consteval) ...

SF F N A SA
1 1 3 10 5

if consteval should not require braces, making this valid: if consteval boom(); else bam();

SF F N A SA
1 3 5 7 4

if consteval is tentatively ready to be voted on to forward to Core, after updating the paper as discussed

SF F N A SA
10 11 1 0 0

@jfbastien jfbastien added needs-revision Paper needs changes before it can proceed EWG-vote-on-me EWG can vote on this labels Oct 8, 2020
@wg21bot
Copy link
Collaborator Author

wg21bot commented Oct 27, 2020

P1938R2 if consteval (Barry Revzin, Daveed Vandevoorde, Richard Smith, Andrew Sutton)

@wg21bot wg21bot removed the needs-revision Paper needs changes before it can proceed label Oct 27, 2020
@jensmaurer jensmaurer modified the milestones: 2020-telecon, 2021-telecon Dec 28, 2020
@jensmaurer jensmaurer added this to Ready for review in CWG Feb 2, 2021
@jfbastien
Copy link
Collaborator

EWG February 2021 polls:

Poll: Forward P1938R2 "if consteval" to Core.

SF F N A SA
20 7 2 0 0

Poll outcome: ✅ consensus.

Salient comments:

  • The number of experts who have been confused by is_constant_evaluated suggests that this is a much safer spelling for the utility.
  • Makes metaprogramming more accessible and its code more readable, better composability.
  • Sad that this wasn’t in C++20.

@jfbastien jfbastien added CWG Core and removed EWG Evolution EWG-vote-on-me EWG can vote on this labels Mar 9, 2021
@jensmaurer jensmaurer moved this from Ready for review to Approved for plenary vote in CWG Mar 22, 2021
@jensmaurer jensmaurer added LWG Library lwg-pending LWG Chair needs to disposition labels Mar 22, 2021
@jensmaurer
Copy link
Member

CWG 2021-03-22: Approved D1938R3.

@JeffGarland
Copy link
Member

LWG review planned for 2021-04-02

@JeffGarland JeffGarland added lwg-fullreview Paper is ready for lwg full group review and removed lwg-pending LWG Chair needs to disposition labels Mar 26, 2021
@jwakely
Copy link
Member

jwakely commented Mar 26, 2021

LWG review planned for 2021-04-02

Change of plan, 2021-04-09 instead.

@JeffGarland
Copy link
Member

Now schedule 2021-04-16

@JeffGarland
Copy link
Member

JeffGarland commented Apr 16, 2021

LWG reviewed and adopted library wording.

https://wiki.edg.com/bin/view/Wg21telecons2021/P1938-20210416

Adopt P1938R3 if consteval for c++23
No objection to unanimous consent

@JeffGarland JeffGarland added tentatively-ready-for-plenary Reviewed between meetings; ready for a vote. and removed lwg-fullreview Paper is ready for lwg full group review tentatively-ready-for-plenary Reviewed between meetings; ready for a vote. labels Apr 16, 2021
@jensmaurer jensmaurer added the straw-poll Straw poll label Apr 23, 2021
@wg21bot
Copy link
Collaborator Author

wg21bot commented Apr 25, 2021

P1938R3 if consteval (Barry Revzin, Daveed Vandevoorde, Richard Smith, Andrew Sutton)

@cor3ntin cor3ntin added plenary-approved Papers approved for inclusion in their target vehicle by plenary vote. C++23 Targeted at C++23 labels Jun 7, 2021
@tkoeppe tkoeppe closed this as completed Jun 15, 2021
@jensmaurer jensmaurer removed this from Approved for plenary vote in CWG Jun 26, 2021
@wg21bot
Copy link
Collaborator Author

wg21bot commented Oct 26, 2021

Adopted 2021-06.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C++23 Targeted at C++23 CWG Core LWG Library plenary-approved Papers approved for inclusion in their target vehicle by plenary vote.
Projects
No open projects
Development

Successfully merging a pull request may close this issue.

7 participants