sammccall added inline comments.
================ Comment at: clang/lib/Sema/SemaOverload.cpp:12880 return Result.getValueOr(QualType()); } ---------------- Wouldn't we be better to handle undeduced type here rather than in the loop? Not sure if it practically matters, but given: `auto f(); double f(int); f(0,0);` the code in this patch will discard auto and yield `double`, while treating this as multiple candidates --> unknown return type seems more correct. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D87350/new/ https://reviews.llvm.org/D87350 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits