[clang] [clang]: fix reference compatibility in overload resolution (PR #133035)
Erich Keane via cfe-commits
cfe-commits at lists.llvm.org
Mon Apr 7 06:53:31 PDT 2025
================
@@ -5450,6 +5450,12 @@ TryReferenceInit(Sema &S, Expr *Init, QualType DeclType,
return ICS;
}
+ if (isRValRef && ICS.UserDefined.ConversionFunction->getReturnType()
----------------
erichkeane wrote:
It isn't clear to me why/if this is the right solution here. Do we have justification here based on the standard that we can put the standards quote here? Also, this is a bit of an awkward place to put this, so it doesn't seem clear to me that this is even the right place.
Unfortunately i'm not particularly knowledgable about this overload code, so hopefully someone more knowledgable comes along.
https://github.com/llvm/llvm-project/pull/133035
More information about the cfe-commits
mailing list