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.

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

[D156032] Implement CWG2137 (list-initialization from objects of the same type) #77638

Closed
ldionne opened this issue Jan 10, 2024 · 2 comments · Fixed by #77768
Closed

[D156032] Implement CWG2137 (list-initialization from objects of the same type) #77638

ldionne opened this issue Jan 10, 2024 · 2 comments · Fixed by #77768
Assignees
@cor3ntin
Labels
clang:frontend Language frontend issues, e.g. anything involving "Sema" phabricator Related to migration from Phabricator

Comments

@ldionne
Copy link
Member

ldionne commented Jan 10, 2024

This issue tracks picking up https://reviews.llvm.org/D156032 from the Phabricator archive, or dropping it in case it has already landed. I want to make sure nothing gets lost.
@ldionne ldionne added the clang Clang issues not falling into any other category label Jan 10, 2024
@EugeneZelenko EugeneZelenko added clang:frontend Language frontend issues, e.g. anything involving "Sema" and removed clang Clang issues not falling into any other category labels Jan 10, 2024
@llvmbot
Copy link
Member

llvmbot commented Jan 10, 2024

@llvm/issue-subscribers-clang-frontend

Author: Louis Dionne (ldionne)

This issue tracks picking up https://reviews.llvm.org/D156032 from the Phabricator archive, or dropping it in case it has already landed. I want to make sure nothing gets lost.

@ldionne ldionne added the phabricator Related to migration from Phabricator label Jan 10, 2024
@cor3ntin
Copy link
Contributor

@MitalAshok Are you still working on that? If so could you create a PR on Github? (Phabricator fell over)
Thanks a lot

@MitalAshok MitalAshok mentioned this issue Jan 11, 2024
Merged
cor3ntin pushed a commit that referenced this issue Jan 19, 2024
@MitalAshok
[SemaCXX] Implement CWG2137 (list-initialization from objects of the …
9247013 
…same type) (#77768)

Closes #77638, #24186

Rebased from <https://reviews.llvm.org/D156032>, see there for more
information.

Implements wording change in [CWG2137](https://wg21.link/CWG2137) in the
first commit.

This also implements an approach to [CWG2311](https://wg21.link/CWG2311)
in the second commit, because too much code that relies on `T{ T_prvalue}` 
being an elision would break. Because that issue is still open and
the CWG issue doesn't provide wording to fix the issue, there may be
different behaviours on other compilers.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@cor3ntin cor3ntin

Labels
clang:frontend Language frontend issues, e.g. anything involving "Sema" phabricator Related to migration from Phabricator
Projects
None yet
Milestone
No milestone
4 participants
@ldionne @cor3ntin @EugeneZelenko @llvmbot