shiboken6: Avoid type name lookup in pointer converters for classes with non-virtual destructors

A lookup by typeid().name() will return the instance type name for
classes with non-virtual destructors; so there is no need to do it.

Change-Id: I911e3133abae37123dc47dda6be479416058668b
Reviewed-by: Christian Tismer <tismer@stackless.com>

1 files changed, 9 insertions, 3 deletions

diff --git a/sources/shiboken6/generator/shiboken/cppgenerator.cpp b/sources/shiboken6/generator/shiboken/cppgenerator.cpp
index 6c9cc5fec..419704153 100644
--- a/sources/shiboken6/generator/shiboken/cppgenerator.cpp
+++ b/sources/shiboken6/generator/shiboken/cppgenerator.cpp
@@ -1598,11 +1598,17 @@ static void writePointerToPythonConverter(TextStream &c,
     c << "auto *pyOut = reinterpret_cast<PyObject *>(Shiboken::BindingManager::instance().retrieveWrapper(cppIn));\n"
         << "if (pyOut) {\n" << indent
         << "Py_INCREF(pyOut);\nreturn pyOut;\n" << outdent
-        << "}\n"
-        << "auto *tCppIn = reinterpret_cast<const " << typeName << R"( *>(cppIn);
-const char *typeName = )";
+        << "}\n";
 
     const QString nameFunc = metaClass->typeEntry()->polymorphicNameFunction();
+    if (nameFunc.isEmpty() && !metaClass->hasVirtualDestructor()) {
+        c << "return Shiboken::Object::newObjectWithHeuristics("
+            << cpythonType << ", const_cast<void *>(cppIn), false);\n";
+        return;
+    }
+
+    c   << "auto *tCppIn = reinterpret_cast<const " << typeName << R"( *>(cppIn);
+const char *typeName = )";
     if (nameFunc.isEmpty())
         c << "typeid(*tCppIn).name();\n";
     else