fix look up of shadow constructor

shadow constructor need to be reconstructed for the base classes

Author: Vipul-Cariappa

lib/CppInterOp/CppInterOp.cpp

@@ -855,8 +855,21 @@ static void GetClassDecls(TCppScope_t klass,
     if (auto* MD = dyn_cast<DeclType>(DI))
       methods.push_back(MD);
     else if (auto* USD = dyn_cast<UsingShadowDecl>(DI))
-      if (auto* MD = dyn_cast<DeclType>(USD->getTargetDecl()))
-        methods.push_back(MD);
+      if (auto* MD = dyn_cast<DeclType>(USD->getTargetDecl())) {
+        if (auto* CUSD = dyn_cast<ConstructorUsingShadowDecl>(DI)) {
+          if (auto* CXXCD =
+                  dyn_cast_or_null<CXXConstructorDecl>(CUSD->getTargetDecl())) {
+            CXXConstructorDecl* Result = getSema().findInheritingConstructor(
+                SourceLocation(), CXXCD, CUSD);
+            // Result is appended to the decls, i.e. CXXRD, iterator
+            // non-shadowed decl will be push_back later
+            // methods.push_back(Result);
+            (void)Result;
+          } else
+            methods.push_back(MD);
+        } else
+          methods.push_back(MD);
+      }
   }
 }
 

unittests/CppInterOp/FunctionReflectionTest.cpp

@@ -95,14 +95,14 @@ TEST(FunctionReflectionTest, GetClassMethods) {
   Cpp::GetClassMethods(Decls[4], methods3);
 
   EXPECT_EQ(methods3.size(), 9);
-  EXPECT_EQ(get_method_name(methods3[0]), "B::B(int n)");
-  EXPECT_EQ(get_method_name(methods3[1]), "inline constexpr B::B(const B &)");
-  EXPECT_EQ(get_method_name(methods3[3]), "inline C::C()");
-  EXPECT_EQ(get_method_name(methods3[4]), "inline constexpr C::C(const C &)");
-  EXPECT_EQ(get_method_name(methods3[5]), "inline constexpr C::C(C &&)");
-  EXPECT_EQ(get_method_name(methods3[6]), "inline C &C::operator=(const C &)");
-  EXPECT_EQ(get_method_name(methods3[7]), "inline C &C::operator=(C &&)");
-  EXPECT_EQ(get_method_name(methods3[8]), "inline C::~C()");
+  EXPECT_EQ(get_method_name(methods3[0]), "inline C::C()");
+  EXPECT_EQ(get_method_name(methods3[1]), "inline constexpr C::C(const C &)");
+  EXPECT_EQ(get_method_name(methods3[2]), "inline constexpr C::C(C &&)");
+  EXPECT_EQ(get_method_name(methods3[3]), "inline C &C::operator=(const C &)");
+  EXPECT_EQ(get_method_name(methods3[4]), "inline C &C::operator=(C &&)");
+  EXPECT_EQ(get_method_name(methods3[5]), "inline C::~C()");
+  EXPECT_EQ(get_method_name(methods3[6]), "inline C::B(int)");
+  EXPECT_EQ(get_method_name(methods3[7]), "inline constexpr C::B(const B &)");
 
   // Should not crash.
   std::vector<Cpp::TCppFunction_t> methods4;