yaxunl added inline comments.
================ Comment at: clang/test/SemaCXX/auto-type-from-cxx.cpp:31-43 +class Abstract{ + public: + void fun(); + virtual void vfun()=0; + void call(){getCaller<Abstract>()(*this);} // expected-note {{in instantiation of function template specialization 'TestDeductionFail::getCaller<TestDeductionFail::Abstract>' requested here}} +}; + ---------------- rsmith wrote: > The end of this test seems unnecessary. Is there something else you want this > test to cover, or can it be simplified? I thought it was necessary to cause clang to emit the diagnostic messages, but you are right that they are unnecessary. I will remove them when committing. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D150212/new/ https://reviews.llvm.org/D150212 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits