================ @@ -1112,6 +1110,7 @@ class PyConcreteAttribute : public BaseTy { /// bindings so such operation always exists). class PyValue { public: + virtual ~PyValue() = default; ---------------- ftynse wrote: Is this still necessary? https://github.com/llvm/llvm-project/pull/69644