sepavloff added inline comments. ================ Comment at: test/SemaTemplate/default-expr-arguments-3.cpp:20 @@ +19,2 @@ + } +} ---------------- Please add the following test to the patch: ``` template<typename T> void f1() { enum class foo { a, b }; struct S { int g1(foo n = foo::a); }; }
template void f1<int>(); ``` It must also compile correctly. https://reviews.llvm.org/D23096 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits