Flesh out test for defaulted key functions a bit more.

llvm-svn: 161243

clang/lib/AST/RecordLayoutBuilder.cpp

@@ -2351,6 +2351,7 @@ RecordLayoutBuilder::ComputeKeyFunction(const CXXRecordDecl *RD) {
     if (MD->hasInlineBody())
       continue;
 
+    // Ignore inline deleted or defaulted functions.
     if (!MD->isUserProvided())
       continue;
 

clang/test/CodeGenCXX/cxx11-vtable-key-function.cpp

@@ -1,11 +1,38 @@
 // RUN: %clang_cc1 %s -triple=x86_64-apple-darwin10 -emit-llvm -o - -std=c++11 | FileCheck %s
 // PR13424
 
+namespace Test1 {
 struct X {
-  virtual ~X() = default;
+  virtual ~X(); // Key function.
-  virtual void f();
+  virtual void f(); // Not a key function.
+};
+
+X::~X() = default;
+
+// Verify that the vtable is emitted.
+// CHECK: @_ZTVN5Test11XE = unnamed_addr constant
+}
+
+namespace Test2 {
+struct X {
+  virtual ~X() = default; // Not a key function.
+  virtual void f(); // Key function.
 };
 
 void X::f() {}
 
-// CHECK: @_ZTV1X = unnamed_addr constant
+// Verify that the vtable is emitted.
+// CHECK: @_ZTVN5Test21XE = unnamed_addr constant
+}
+
+namespace Test3 {
+struct X {
+  virtual ~X() = delete; // Not a key function.
+  virtual void f(); // Key function.
+};
+
+void X::f() {}
+
+// Verify that the vtable is emitted.
+// CHECK: @_ZTVN5Test31XE = unnamed_addr constant
+}