[clang]: fix overload resolution in case of bind rvalue ref to const lvalue ref

[changkhothuychung]

fix #132600

[llvmbot]

@llvm/pr-subscribers-clang

Author: Nhat Nguyen (changkhothuychung)

Changes

fix #132600

---

Full diff: https://github.com/llvm/llvm-project/pull/133035.diff

1 Files Affected:

• (modified) clang/lib/Sema/SemaOverload.cpp (+1-1)

diff --git a/clang/lib/Sema/SemaOverload.cpp b/clang/lib/Sema/SemaOverload.cpp
index 6d8006b35dcf4..28843f378c686 100644
--- a/clang/lib/Sema/SemaOverload.cpp
+++ b/clang/lib/Sema/SemaOverload.cpp
@@ -5419,7 +5419,7 @@ TryReferenceInit(Sema &S, Expr *Init, QualType DeclType,
   //   the argument expression. Any difference in top-level
   //   cv-qualification is subsumed by the initialization itself
   //   and does not constitute a conversion.
-  ICS = TryImplicitConversion(S, Init, T1, SuppressUserConversions,
+  ICS = TryImplicitConversion(S, Init, DeclType, SuppressUserConversions,
                               AllowedExplicit::None,
                               /*InOverloadResolution=*/false,
                               /*CStyle=*/false,

[github-actions]

✅ With the latest revision this PR passed the C/C++ code formatter.

[changkhothuychung]

the failing test in clang/test/SemaCXX/copy-assignment.cpp seems like to be another clang bug - https://godbolt.org/z/79bWMscx9