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
