]> granicus.if.org Git - clang/commitdiff
Discussing with dgregor we decided that we should not force the emission of
authorRafael Espindola <rafael.espindola@gmail.com>
Wed, 24 Mar 2010 22:43:31 +0000 (22:43 +0000)
committerRafael Espindola <rafael.espindola@gmail.com>
Wed, 24 Mar 2010 22:43:31 +0000 (22:43 +0000)
implicit methods on explicit template instantiation definitions. As a
consequence, we should emit them at every use, even if we see a explicit
template instantiation declaration.

This is already the current behaviour, but it is good to test for that :-)

git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@99443 91177308-0d34-0410-b5e6-96231b3b80d8

test/CodeGenCXX/template-instantiation.cpp [moved from test/CodeGenCXX/PR6677.cpp with 59% similarity]

similarity index 59%
rename from test/CodeGenCXX/PR6677.cpp
rename to test/CodeGenCXX/template-instantiation.cpp
index 8d168f110608039b1429f0bc23a32692d6415943..9e0593998bb3a4c3c378e541f2d04074d23c3b3c 100644 (file)
@@ -3,6 +3,10 @@
 // CHECK-NOT: @_ZTVN5test118stdio_sync_filebufIwEE = constant
 // CHECK: @_ZTVN5test018stdio_sync_filebufIwEE = constant
 
+// CHECK: define linkonce_odr void @_ZN5test21CIiEC1Ev(
+// CHECK: define linkonce_odr void @_ZN5test21CIiE6foobarIdEEvT_(
+// CHECK: define available_externally void @_ZN5test21CIiE6zedbarEd(
+
 namespace test0 {
   struct  basic_streambuf   {
     virtual       ~basic_streambuf();
@@ -31,3 +35,28 @@ namespace test1 {
   // Just a declaration should not force the vtable to be emitted.
   template<> void stdio_sync_filebuf<wchar_t>::xsgetn();
 }
+
+namespace test2 {
+  template<typename T1>
+  class C {
+    virtual ~C();
+    void zedbar(double) {
+    }
+    template<typename T2>
+    void foobar(T2 foo) {
+    }
+  };
+  extern template class C<int>;
+  void g() {
+    // The extern template declaration should not prevent us from producing
+    // the implicit constructor (test at the top).
+    C<int> a;
+
+    // or foobar(test at the top).
+    a.foobar(0.0);
+
+    // But it should prevent zebbar
+    // (test at the top).
+    a.zedbar(0.0);
+  }
+}