https://github.com/spaits created 
https://github.com/llvm/llvm-project/pull/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.)

From e298ac39d1629c710b4b8ad2e15d22f6e8ecd0eb Mon Sep 17 00:00:00 2001
From: Gabor Spaits <gabor.spa...@hightec-rt.com>
Date: Sat, 22 Jun 2024 22:35:22 +0200
Subject: [PATCH] Simplify release notes and remove irrelevant comment

---
 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 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)

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to