llvmbot wrote:
<!--LLVM PR SUMMARY COMMENT--> @llvm/pr-subscribers-clang Author: Gábor Spaits (spaits) <details> <summary>Changes</summary> As discussed before with @<!-- -->cor3ntin before (https://github.com/llvm/llvm-project/pull/94752) here is the simplification of the release note written for the previously mentioned PR and the removal of a comment that is no longer useful. (Sorry for creating this PR this late.) --- Full diff: https://github.com/llvm/llvm-project/pull/96407.diff 2 Files Affected: - (modified) clang/docs/ReleaseNotes.rst (-3) - (modified) clang/lib/Sema/SemaInit.cpp (-3) ``````````diff diff --git a/clang/docs/ReleaseNotes.rst b/clang/docs/ReleaseNotes.rst index 9c8f8c4a4fbaf..d32a85f2fb698 100644 --- a/clang/docs/ReleaseNotes.rst +++ b/clang/docs/ReleaseNotes.rst @@ -224,9 +224,6 @@ C++20 Feature Support ``<expected>`` from libstdc++ to work correctly with Clang. - User defined constructors are allowed for copy-list-initialization with CTAD. - The example code for deduction guides for std::map in - (`cppreference <https://en.cppreference.com/w/cpp/container/map/deduction_guides>`_) - will now work. (#GH62925). C++23 Feature Support diff --git a/clang/lib/Sema/SemaInit.cpp b/clang/lib/Sema/SemaInit.cpp index f820db5233f53..a27f97ae739f5 100644 --- a/clang/lib/Sema/SemaInit.cpp +++ b/clang/lib/Sema/SemaInit.cpp @@ -11041,9 +11041,6 @@ QualType Sema::DeduceTemplateSpecializationFromInitializer( // C++ [over.best.ics]p4: // When [...] the constructor [...] is a candidate by // - [over.match.copy] (in all cases) - // FIXME: The "second phase of [over.match.list] case can also - // theoretically happen here, but it's not clear whether we can - // ever have a parameter of the right type. if (TD) { SmallVector<Expr *, 8> TmpInits; for (Expr *E : Inits) `````````` </details> https://github.com/llvm/llvm-project/pull/96407 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
