From de88b2cb16af4bba659d0bb2ddf10bda681ec84d Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?G=C3=A1bor=20Spaits?= Date: Sat, 6 Jul 2024 15:22:51 +0200 Subject: [PATCH] [Clang] Simplify release notes and remove irrelevant comment (#96407) 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.) Co-authored-by: Gabor Spaits --- clang/docs/ReleaseNotes.rst | 3 --- clang/lib/Sema/SemaInit.cpp | 3 --- 2 files changed, 6 deletions(-) diff --git a/clang/docs/ReleaseNotes.rst b/clang/docs/ReleaseNotes.rst index 39187078d5786..edc932efd9416 100644 --- a/clang/docs/ReleaseNotes.rst +++ b/clang/docs/ReleaseNotes.rst @@ -236,9 +236,6 @@ C++20 Feature Support ```` 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 `_) - will now work. (#GH62925). C++23 Feature Support diff --git a/clang/lib/Sema/SemaInit.cpp b/clang/lib/Sema/SemaInit.cpp index 64e43ded0961e..41753a1661ace 100644 --- a/clang/lib/Sema/SemaInit.cpp +++ b/clang/lib/Sema/SemaInit.cpp @@ -9811,9 +9811,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 TmpInits; for (Expr *E : Inits)