[PATCH] D74009: [clang] Improve diagnostic note for implicit conversions that are disallowed because they involve more than one user-defined conversion.

Logan Smith via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Feb 5 16:56:34 PST 2020


logan-5 added a comment.

In D74009#1858353 <https://reviews.llvm.org/D74009#1858353>, @rsmith wrote:

> Have you looked into whether you can defer the check for conversion via multiple user-defined conversions until we come to call `CompleteNonViableCandidate` after finding no viable functions?


That's a good idea, and does seem more efficient than checking for multiple conversions every time. I'll look into that, and also split this patch into two as @jkorous suggested.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D74009/new/

https://reviews.llvm.org/D74009





More information about the cfe-commits mailing list