https://github.com/hax0kartik updated https://github.com/llvm/llvm-project/pull/177534
>From 83b72437756b9678a0309aedad641925ea2802b9 Mon Sep 17 00:00:00 2001 From: hax0kartik <[email protected]> Date: Fri, 23 Jan 2026 12:05:46 +0530 Subject: [PATCH] [clang] Fix a crash when default argument is passed to an explicit object parameter This is an invalid usecase and we should mark it as such after outputting a Diagnostic. --- clang/docs/ReleaseNotes.rst | 1 + clang/lib/Sema/SemaDeclCXX.cpp | 1 + clang/test/SemaCXX/cxx2b-deducing-this.cpp | 24 ++++++++++++++++++++++ 3 files changed, 26 insertions(+) diff --git a/clang/docs/ReleaseNotes.rst b/clang/docs/ReleaseNotes.rst index 791212dafd342..4cd25b767320a 100644 --- a/clang/docs/ReleaseNotes.rst +++ b/clang/docs/ReleaseNotes.rst @@ -173,6 +173,7 @@ Bug Fixes to Attribute Support Bug Fixes to C++ Support ^^^^^^^^^^^^^^^^^^^^^^^^ - Fixed a crash when instantiating ``requires`` expressions involving substitution failures in C++ concepts. (#GH176402) +- Fixed a crash when a default argument is passed to an explicit object parameter. (#GH176639) Bug Fixes to AST Handling ^^^^^^^^^^^^^^^^^^^^^^^^^ diff --git a/clang/lib/Sema/SemaDeclCXX.cpp b/clang/lib/Sema/SemaDeclCXX.cpp index c96189172a010..5837ecd6b9163 100644 --- a/clang/lib/Sema/SemaDeclCXX.cpp +++ b/clang/lib/Sema/SemaDeclCXX.cpp @@ -11600,6 +11600,7 @@ void Sema::CheckExplicitObjectMemberFunction(Declarator &D, Diag(ExplicitObjectParam->getLocation(), diag::err_explicit_object_default_arg) << ExplicitObjectParam->getSourceRange(); + D.setInvalidType(); } if (D.getDeclSpec().getStorageClassSpec() == DeclSpec::SCS_static || diff --git a/clang/test/SemaCXX/cxx2b-deducing-this.cpp b/clang/test/SemaCXX/cxx2b-deducing-this.cpp index a9e31c3d06676..3e4bf509a7287 100644 --- a/clang/test/SemaCXX/cxx2b-deducing-this.cpp +++ b/clang/test/SemaCXX/cxx2b-deducing-this.cpp @@ -1400,3 +1400,27 @@ a void Bar(this int) { // expected-note {{candidate function}} } } + +namespace GH176639 { + +struct S { + void operator()(this S =) // expected-error {{the explicit object parameter cannot have a default argument}} + // expected-error@-1 {{expected ';' at end of declaration list}} + // expected-error@-2 {{expected expression}} +}; + +void foo() { + S s{}; + s(0); // expected-error {{no matching function for call to object of type 'S'}} +} + +struct S2 { + void operator()(this S2 = S2 {}){} // expected-error {{the explicit object parameter cannot have a default argument}} +}; + +void foo2() { + S2 s{}; + s(0); // expected-error {{no matching function for call to object of type 'S2'}} +} + +} _______________________________________________ cfe-commits mailing list [email protected] https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
