Skip to content

__cpp_explicit_this_parameter not set in clang-18 #82780

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
Closed
mpusz opened this issue Feb 23, 2024 · 1 comment · Fixed by #107451
Closed

__cpp_explicit_this_parameter not set in clang-18 #82780

mpusz opened this issue Feb 23, 2024 · 1 comment · Fixed by #107451
Labels
clang:driver 'clang' and 'clang++' user-facing binaries. Not 'clang-cl'

Comments

@mpusz
Copy link

mpusz commented Feb 23, 2024

It seems that clang-18 supports explicit object parameter (deducing this) however the feature test macro is not set: https://godbolt.org/z/j6za1evK9
@EugeneZelenko EugeneZelenko added clang:driver 'clang' and 'clang++' user-facing binaries. Not 'clang-cl' and removed new issue labels Feb 23, 2024
@llvmbot
Copy link
Member

llvmbot commented Feb 23, 2024

@llvm/issue-subscribers-clang-driver

Author: Mateusz Pusz (mpusz)

It seems that clang-18 [supports explicit object parameter (deducing this)](https://en.cppreference.com/w/cpp/compiler_support/23) however the feature test macro is not set: https://godbolt.org/z/j6za1evK9

@mpusz mpusz mentioned this issue Jul 4, 2024
Open
@cor3ntin cor3ntin mentioned this issue Sep 5, 2024
Merged
cor3ntin added a commit to cor3ntin/llvm-project that referenced this issue Sep 5, 2024
@cor3ntin
[Clang] Set __cpp_explicit_this_parameter
65a8fbf 
There are not a lot of outstanding known issues
with deducing this (besides llvm#95112), so it
seems reasonable to claim full support.

Fixes llvm#82780
cor3ntin added a commit that referenced this issue Dec 18, 2024
@cor3ntin
[Clang] Set __cpp_explicit_this_parameter (#107451)
72e58e0 
There are not a lot of outstanding known issues
with deducing this (besides #95112), so it
seems reasonable to claim full support.

Fixes #82780
@frederick-vs-ja frederick-vs-ja marked this as a duplicate of #113174 Jan 20, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
clang:driver 'clang' and 'clang++' user-facing binaries. Not 'clang-cl'
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[Clang] Set __cpp_explicit_this_parameter cor3ntin/llvm-project
3 participants
@mpusz @EugeneZelenko @llvmbot