Re: [PATCH] D17215: [Sema] Fix PR14211 Crash for explicit instantiation of overloaded template function within class template

[John McCall via cfe-commits]

rjmccall added inline comments.

================
Comment at: lib/Sema/SemaTemplate.cpp:7845
@@ -7842,1 +7844,3 @@
+          } else
+            NonTemplateMatch = Method;
         }
----------------
Could you add an assertion here that NonTemplateMatch is still null?  That 
should definitely never trip.

Hmm, actually, it might trip in invalid code; you should include a test case 
like
  template <class T, class U> class A { void foo(T) {} void foo(U) {} };
  template void A<int, int>::foo(int);


http://reviews.llvm.org/D17215



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits