================
@@ -6567,7 +6567,8 @@ SemaCodeCompletion::ProduceCallSignatureHelp(Expr *Fn, 
ArrayRef<Expr *> Args,
                                       CandidateSet,
                                       /*ExplicitArgs=*/nullptr,
                                       /*SuppressUserConversions=*/false,
-                                      /*PartialOverloading=*/true);
+                                      /*PartialOverloading=*/true,
+                                      /*FirstArgumentIsBase=*/true);
----------------
timon-ul wrote:

I had some time to think about this more and I think this is actually not that 
nice. Essentially with this we are kinda telling `AddFunctionCandidates` that 
this is always static (or more specifically, that the first argument is always 
base, which just isn't true) and depend on the function first handling the non 
static way. I guess if there is some good reason why this would be hard to do 
or still unneccessary I am willing to keep it this way, but please have a look 
if we can handle this more gracefully.

https://github.com/llvm/llvm-project/pull/234918
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to