Skip to content

CWG2563 Initialization of coroutine result object #1414

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.

Sign up for GitHub

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

Jump to bottom
Open
jensmaurer opened this issue Jan 5, 2023 · 1 comment
Open

CWG2563 Initialization of coroutine result object #1414

jensmaurer opened this issue Jan 5, 2023 · 1 comment
Labels
EWG Evolution needs-revision Paper needs changes before it can proceed

Comments

@jensmaurer
Copy link
Member

CWG solicits EWG input on the handling of coroutine result objects in the presence of conversions and non-movable and non-copyable types.

See CWG2563 for details.
@jensmaurer jensmaurer added the EWG Evolution label Jan 5, 2023
@jensmaurer jensmaurer added this to the 2023-telecon milestone Jan 25, 2023
@erichkeane
Copy link
Collaborator

EWG discussed this in the February 6th, 2023 afternoon session in Issaquah.

The following poll was taken:

EWG agrees with Gor & Lewis' explained semantics to CWG2563, and instructs them to enpanel a CWG expert to assist in turning this into wording, and bring it back to EWG.

SF F N A SA
9 5 1 0 0

Result: Consensus

@erichkeane erichkeane added the needs-revision Paper needs changes before it can proceed label Feb 7, 2023
@bcardosolopes bcardosolopes mentioned this issue Feb 14, 2023
Closed
bcardosolopes added a commit to llvm/llvm-project that referenced this issue Mar 22, 2023
@bcardosolopes
[Coroutines] Implement fix for cwg2563 issue and enable RVO under cer…
fa0d4e1 
…tain conditions

- The cwg2563 issue is fixed by delaying GRO initialization only when the types
  mismatch between GRO and function return.
- When the types match directly initialize, which indirectly enables RVO to
  kick in, partially restores behavior introduced in
  https://reviews.llvm.org/D117087.
- Add entry to release notes.

Background:
#56532
https://cplusplus.github.io/CWG/issues/2563.html
cplusplus/papers#1414

Differential Revision: https://reviews.llvm.org/D145641
@jensmaurer jensmaurer removed this from the 2023-telecon milestone Mar 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
EWG Evolution needs-revision Paper needs changes before it can proceed
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@erichkeane @jensmaurer