================
@@ -1115,5 +1115,21 @@ searchConstructorsInForwardingFunction(const
FunctionDecl *FD) {
return Result;
}
+ArrayRef<const CXXConstructorDecl *>
----------------
timon-ul wrote:
I thought about this return type for a while, the main issue I personally had
with it is that the caller `findIndirectConstructors` turns this back into a
vector. I mean we expect these vectors to have 1 element only so a copy is
probably pretty cheap (also I am unsure how much more overhead it is than
creating an `ArrayRef`), that being said I think using this is fine, but then
`findIndirectConstructors`, which is the same use case as the other callers
just with 1 more layer of function in between should also just return a
reference to make this more coherent. (Sorry for commenting here and not where
I expect the change oops)
https://github.com/llvm/llvm-project/pull/199480
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
