dblaikie added inline comments.
================
Comment at: clang/test/SemaTemplate/class-template-id.cpp:12-14
+ return ptr2; // expected-error{{cannot initialize return object of type
'A<int, FLOAT> *' (aka 'A<int, float> *') with an lvalue of type 'const A<int>
*'}}
else {
+ return ptr3; // expected-error{{cannot initialize return object of type
'A<int, FLOAT> *' (aka 'A<int, float> *') with an lvalue of type 'A<int,
double> *'}}
----------------
FWIW, looks like this was a reduction in quality - printing the value of the
default argument in these error messages, when omitting it was probably
desirable?
& doesn't seem to be a necessary consequence of the patch description's goals.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D103040/new/
https://reviews.llvm.org/D103040
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
