jkorous added a reviewer: jkorous.
jkorous added a comment.
Herald added a subscriber: dexonsmith.

Hi @logan-5!
I suggest you split the patch into two smaller ones so it is easier to review.

1. A NFC patch with refactoring of the interface (`bool` -> 
`UserDefinedConversionsKind`).
2. Patch that introduces and uses 
`diag::note_ovl_candidate_bad_user_defined_conv`.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D74009



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

Reply via email to