diff --git a/clang/docs/ReleaseNotes.rst b/clang/docs/ReleaseNotes.rst index 2ecd19bdc3944..7e78e7eb9ec0a 100644 --- a/clang/docs/ReleaseNotes.rst +++ b/clang/docs/ReleaseNotes.rst @@ -760,6 +760,7 @@ Bug Fixes to C++ Support missing placeholder return type. (#GH78694) - Fixed a bug where bounds of partially expanded pack indexing expressions were checked too early. (#GH116105) - Fixed an assertion failure caused by using ``consteval`` in condition in consumed analyses. (#GH117385) +- Fix a crash caused by incorrect argument position in merging deduced template arguments. (#GH113659) Bug Fixes to AST Handling ^^^^^^^^^^^^^^^^^^^^^^^^^ diff --git a/clang/lib/Sema/SemaTemplateDeduction.cpp b/clang/lib/Sema/SemaTemplateDeduction.cpp index 62a0c30d99502..fad20b37a7d9a 100644 --- a/clang/lib/Sema/SemaTemplateDeduction.cpp +++ b/clang/lib/Sema/SemaTemplateDeduction.cpp @@ -376,7 +376,7 @@ checkDeducedTemplateArguments(ASTContext &Context, for (TemplateArgument::pack_iterator XA = X.pack_begin(), XAEnd = X.pack_end(), YA = Y.pack_begin(), YAEnd = Y.pack_end(); - XA != XAEnd; ++XA, ++YA) { + XA != XAEnd; ++XA) { if (YA != YAEnd) { TemplateArgument Merged = checkDeducedTemplateArguments( Context, DeducedTemplateArgument(*XA, X.wasDeducedFromArrayBound()), @@ -384,6 +384,7 @@ checkDeducedTemplateArguments(ASTContext &Context, if (Merged.isNull() && !(XA->isNull() && YA->isNull())) return DeducedTemplateArgument(); NewPack.push_back(Merged); + ++YA; } else { NewPack.push_back(*XA); } diff --git a/clang/test/SemaTemplate/template-args-deduction-aggregates.cpp b/clang/test/SemaTemplate/template-args-deduction-aggregates.cpp new file mode 100644 index 0000000000000..afe78d0873bb3 --- /dev/null +++ b/clang/test/SemaTemplate/template-args-deduction-aggregates.cpp @@ -0,0 +1,12 @@ +// RUN: %clang_cc1 -fsyntax-only -verify %s -std=c++20 + +// expected-no-diagnostics + +namespace GH113659 { +template struct S {}; +struct T {}; +struct U {}; + +template struct B : S, Args... {}; +B b{S{}}; +}