When laying out vtables for virtual bases in construction vtables, we need to check if the vtable is a primary base in the layout class.

llvm-svn: 97402

1 files changed, 14 insertions, 7 deletions

diff --git a/clang/test/CodeGenCXX/vtable-layout.cpp b/clang/test/CodeGenCXX/vtable-layout.cpp
index dce7461e042..615ab6f5645 100644
--- a/clang/test/CodeGenCXX/vtable-layout.cpp
+++ b/clang/test/CodeGenCXX/vtable-layout.cpp
@@ -926,6 +926,13 @@ namespace Test24 {
 
 // Another construction vtable test.
 
+struct A {
+  virtual void f();
+};
+
+struct B : virtual A { };
+struct C : virtual A { };
+
 // CHECK:      Vtable for 'Test24::D' (10 entries).
 // CHECK-NEXT:    0 | vbase_offset (0)
 // CHECK-NEXT:    1 | vcall_offset (0)
@@ -952,13 +959,13 @@ namespace Test24 {
 // CHECK-NEXT:        -- (Test24::B, 0) vtable address --
 // CHECK-NEXT:    4 | void Test24::A::f()
 
-struct A {
-  virtual void f();
-};
-
-struct B : virtual A { };
-struct C : virtual A { };
-
+// CHECK:      Construction vtable for ('Test24::C', 8) in 'Test24::D' (9 entries).
+// CHECK-NEXT:    0 | vbase_offset (-8)
+// CHECK-NEXT:    1 | vcall_offset (-8)
+// CHECK-NEXT:    2 | offset_to_top (0)
+// CHECK-NEXT:    3 | Test24::C RTTI
+// CHECK-NEXT:        -- (Test24::A, 8) vtable address --
+// CHECK-NEXT:        -- (Test24::C, 8) vtable address --
 struct D : B, C {
   virtual void f();
 };