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

Clang C++23 Feature P0847R7 Deducing This #59619

Closed
8 of 11 tasks
cor3ntin opened this issue Dec 20, 2022 · 9 comments
Closed
8 of 11 tasks

Clang C++23 Feature P0847R7 Deducing This #59619

cor3ntin opened this issue Dec 20, 2022 · 9 comments
Assignees
Labels
c++23 clang:frontend Language frontend issues, e.g. anything involving "Sema" metabug Issue to collect references to a group of similar or related issues.

Comments

@cor3ntin
Copy link
Contributor

cor3ntin commented Dec 20, 2022

https://wg21.link/P0847

Creating that to track progress and signal that I'm working on it

Core issues

@cor3ntin cor3ntin added the c++23 label Dec 20, 2022
@cor3ntin cor3ntin self-assigned this Dec 20, 2022
@llvmbot
Copy link
Collaborator

llvmbot commented Dec 20, 2022

@llvm/issue-subscribers-c-2b

@royjacobson
Copy link
Contributor

Good luck! 🥇

Did you think yet about what to do with mangling? I tried asking on itanium-cxx-abi/cxx-abi#148 a while ago but got no answer.

@cor3ntin
Copy link
Contributor Author

Nah.
It will be an issue because, unfortunately

struct S {
  static void f(S);
  void f(this S);
};

is valid. I suspect that's why msvc has different mangling
I'm hoping someone can solve that before i get there :)

@frederick-vs-ja
Copy link
Contributor

Nah. It will be an issue because, unfortunately

struct S {
  static void f(S);
  void f(this S);
};

is valid. I suspect that's why msvc has different mangling I'm hoping someone can solve that before i get there :)

CWG2692 would make this invalid. I guess we can just implement the proposed resolution.

@cor3ntin
Copy link
Contributor Author

@frederick-vs-ja Unfortunately, the adopted resolution reinforce that this should be valid

@abbator
Copy link

abbator commented Aug 2, 2023

Are there any updates on progress here? Very interested in this feature and eagerly awaiting support for it by Clang.

@cor3ntin
Copy link
Contributor Author

cor3ntin commented Aug 2, 2023

Are there any updates on progress here? Very interested in this feature and eagerly awaiting support for it by Clang.

PR here https://reviews.llvm.org/D140828 .

I'm hoping we can work on review over the next few weeks. Clang 18 sounds like a realistic release target. No promise!

@Endilll
Copy link
Contributor

Endilll commented Jul 6, 2024

CWG2692 is resolved by P2797R0 and is implemented in #93430, which declares the feature complete in the release notes.
Corentin suggested to me offline that this should be closed. Unimplemented Core issues can be handled as usual when they got resolved.

@Endilll Endilll closed this as completed Jul 6, 2024
@Endilll Endilll added the clang:frontend Language frontend issues, e.g. anything involving "Sema" label Jul 6, 2024
@EugeneZelenko EugeneZelenko added the metabug Issue to collect references to a group of similar or related issues. label Jul 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
c++23 clang:frontend Language frontend issues, e.g. anything involving "Sema" metabug Issue to collect references to a group of similar or related issues.
Projects
None yet
Development

No branches or pull requests

7 participants