================ @@ -8054,8 +8056,9 @@ bool Sema::CheckNonDependentConversions( bool HasThisConversion = Method && !isa<CXXConstructorDecl>(Method); unsigned ThisConversions = HasThisConversion ? 1 : 0; - Conversions = - CandidateSet.allocateConversionSequences(ThisConversions + Args.size()); + Conversions = Conversions.empty() ? CandidateSet.allocateConversionSequences( ---------------- erichkeane wrote:
Rather than Ternary, I think this should just be; `if (Conversions.empty()) Conversions = CandidateSet....` https://github.com/llvm/llvm-project/pull/122423 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits