[cxx-interop] Fix lifetime-operation-methods.swift in optimized mode

Author: egorzhdan

test/Interop/Cxx/foreign-reference/Inputs/lifetime-operation-methods.h

@@ -62,18 +62,18 @@ struct CRTPDerived : CRTPBase<CRTPDerived> {
 struct VirtualRetainRelease {
   int value;
   mutable int refCount = 1;
+  mutable bool calledBase = false;
   VirtualRetainRelease(int value) : value(value) {}
 
-  virtual void doRetainVirtual() const { refCount++; }
-  virtual void doReleaseVirtual() const { refCount--; }
+  virtual void doRetainVirtual() const { refCount++; calledBase = true; }
+  virtual void doReleaseVirtual() const { refCount--; calledBase = true; }
   virtual ~VirtualRetainRelease() = default;
 } SWIFT_SHARED_REFERENCE(.doRetainVirtual, .doReleaseVirtual);
 
 struct DerivedVirtualRetainRelease : VirtualRetainRelease {
   DerivedVirtualRetainRelease(int value) : VirtualRetainRelease(value) {}
 
-  mutable bool calledDerived = false;
-  void doRetainVirtual() const override { refCount++; calledDerived = true; }
+  void doRetainVirtual() const override { refCount++; }
   void doReleaseVirtual() const override { refCount--; }
 };
 

test/Interop/Cxx/foreign-reference/lifetime-operation-methods.swift

@@ -63,7 +63,7 @@ LifetimeMethodsTestSuite.test("virtual retain/release") {
 LifetimeMethodsTestSuite.test("overridden virtual retain/release") {
   let a = DerivedVirtualRetainRelease(456)
   expectEqual(a.value, 456)
-  expectTrue(a.calledDerived)
+  expectFalse(a.calledBase) // in optimized builds, we might not call retain/release at all
   expectTrue(a.refCount > 0)
   expectTrue(a.refCount < 10) // optimizations would affect the exact number
 }