hokein marked an inline comment as done. hokein added inline comments.
================ Comment at: clang/test/CXX/temp/temp.decls/temp.variadic/fixed-expansion.cpp:129 + S<int, int, double> &s1 = f({}, 0, 0.0); // expected-error {{no matching function}} \ + expected-error {{non-const lvalue reference to type 'S<int, int, double>' cannot bind to a value of unrelated type 'int'}} } ---------------- the secondary diagnostic is technically correct, but I don't quite like it, it is confusing, ok to leave it as-is? or fix that before landing this patch? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D82657/new/ https://reviews.llvm.org/D82657 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits