r192786 - Remove a workaround for PR16406 from the vtables-virtual-inheritance test

Timur Iskhodzhanov timurrrr at google.com
Wed Oct 16 04:38:29 PDT 2013


Author: timurrrr
Date: Wed Oct 16 06:38:29 2013
New Revision: 192786

URL: http://llvm.org/viewvc/llvm-project?rev=192786&view=rev
Log:
Remove a workaround for PR16406 from the vtables-virtual-inheritance test

Modified:
    cfe/trunk/test/CodeGenCXX/microsoft-abi-vtables-virtual-inheritance.cpp

Modified: cfe/trunk/test/CodeGenCXX/microsoft-abi-vtables-virtual-inheritance.cpp
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/test/CodeGenCXX/microsoft-abi-vtables-virtual-inheritance.cpp?rev=192786&r1=192785&r2=192786&view=diff
==============================================================================
--- cfe/trunk/test/CodeGenCXX/microsoft-abi-vtables-virtual-inheritance.cpp (original)
+++ cfe/trunk/test/CodeGenCXX/microsoft-abi-vtables-virtual-inheritance.cpp Wed Oct 16 06:38:29 2013
@@ -42,7 +42,6 @@ struct C: virtual A {
 
   // MANGLING-DAG: @"\01??_7C@@6B@"
 
-  ~C();  // Currently required to have correct record layout, see PR16406
   virtual void f();
 };
 
@@ -148,7 +147,7 @@ struct X: virtual C {
 
   // TEST4: VFTable for 'A' in 'C' in 'Test4::X' (2 entries).
   // TEST4-NEXT: 0 | void C::f()
-  // TEST4-NEXT: [this adjustment: 12 non-virtual]
+  // TEST4-NEXT: [this adjustment: 8 non-virtual]
   // TEST4-NEXT: 1 | void A::z()
 
   // TEST4-NOT: VFTable indices for 'Test4::X'
@@ -156,7 +155,7 @@ struct X: virtual C {
   // MANGLING-DAG: @"\01??_7X at Test4@@6B@"
 
   // Also check the mangling of the thunk.
-  // MANGLING-DAG: define weak x86_thiscallcc void @"\01?f at C@@WPPPPPPPE at AEXXZ"
+  // MANGLING-DAG: define weak x86_thiscallcc void @"\01?f at C@@WPPPPPPPI at AEXXZ"
 };
 
 X x;
@@ -215,11 +214,11 @@ struct X : C {
 struct Y : virtual X {
   // TEST7: VFTable for 'A' in 'C' in 'Test7::X' in 'Test7::Y' (2 entries).
   // TEST7-NEXT: 0 | void C::f()
-  // TEST7-NEXT:     [this adjustment: 12 non-virtual]
+  // TEST7-NEXT:     [this adjustment: 8 non-virtual]
   // TEST7-NEXT: 1 | void A::z()
 
   // TEST7: Thunks for 'void C::f()' (1 entry).
-  // TEST7-NEXT: 0 | this adjustment: 12 non-virtual
+  // TEST7-NEXT: 0 | this adjustment: 8 non-virtual
 
   // TEST7-NOT: VFTable indices for 'Test7::Y'
 
@@ -336,8 +335,6 @@ struct W : Z, D, virtual A, virtual B {
 W w;
 
 struct T : Z, D, virtual A, virtual B {
-  ~T();  // Currently required to have correct record layout, see PR16406
-
   // TEST9-T: VFTable for 'Test9::Y' in 'Test9::Z' in 'Test9::T' (1 entries).
   // TEST9-T-NEXT: 0 | void Test9::T::h()
 
@@ -357,15 +354,15 @@ struct T : Z, D, virtual A, virtual B {
 
   // TEST9-T: VFTable for 'A' in 'D' in 'Test9::T' (2 entries).
   // TEST9-T-NEXT: 0 | void Test9::T::f()
-  // TEST9-T-NEXT:     [this adjustment: -16 non-virtual]
+  // TEST9-T-NEXT:     [this adjustment: -8 non-virtual]
   // TEST9-T-NEXT: 1 | void Test9::T::z()
-  // TEST9-T-NEXT:     [this adjustment: -16 non-virtual]
+  // TEST9-T-NEXT:     [this adjustment: -8 non-virtual]
 
   // TEST9-T: Thunks for 'void Test9::T::f()' (1 entry).
-  // TEST9-T-NEXT: 0 | this adjustment: -16 non-virtual
+  // TEST9-T-NEXT: 0 | this adjustment: -8 non-virtual
 
   // TEST9-T: Thunks for 'void Test9::T::z()' (1 entry).
-  // TEST9-T-NEXT: 0 | this adjustment: -16 non-virtual
+  // TEST9-T-NEXT: 0 | this adjustment: -8 non-virtual
 
   // TEST9-T: VFTable indices for 'Test9::T' (4 entries).
   // TEST9-T-NEXT: via vfptr at offset 0





More information about the cfe-commits mailing list