Merge pull request swiftlang#582 from Teemperor/AddDestructorCallOverriden

[lldb][NFC] Fix call-overridden-method test by adding virtual destructor

Author: fredriss

lldb/packages/Python/lldbsuite/test/commands/expression/call-overridden-method/TestCallOverriddenMethod.py

@@ -45,8 +45,6 @@ def test_call_on_base(self):
         # Test calling the base class.
         self.expect("expr realbase.foo()", substrs=["= 1"])
 
-    @skipIfLinux # Returns wrong result code on some platforms.
-    @expectedFailureAll(archs=['arm64', 'arm64e'], bugnumber="<rdar://problem/56828233>")
     def test_call_on_derived(self):
         """Test calls to overridden methods in derived classes."""
         self.build()
@@ -64,7 +62,6 @@ def test_call_on_derived(self):
         # a vtable entry that does not exist in the compiled program).
         self.expect("expr d.foo()", substrs=["= 2"])
 
-    @skipIfLinux # Calling constructor causes SIGABRT
     @expectedFailureAll(oslist=["windows"], bugnumber="llvm.org/pr43707")
     def test_call_on_temporary(self):
         """Test calls to overridden methods in derived classes."""

lldb/packages/Python/lldbsuite/test/commands/expression/call-overridden-method/main.cpp

@@ -6,6 +6,7 @@ class Base {
 
 class Derived : public Base {
 public:
+  virtual ~Derived() {}
   virtual int foo() { return 2; }
 };