aaron.ballman added a comment. In D67113#2243451 <https://reviews.llvm.org/D67113#2243451>, @aaronpuchert wrote:
> Maybe this is to trivial for a review. The comment on > `StandardConversionSequence::Third` in clang/Sema/Overload.h says > >> The third conversion can be a qualification conversion or a function >> conversion. There may be a deeper issue here in that there are four standard conversion sequences, not three: http://eel.is/c++draft/conv#1, but otherwise this change seems correct to me (function conversion is certainly not in the first group). Do you have a test case which can exercise this change? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D67113/new/ https://reviews.llvm.org/D67113 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
