db7 wrote:

> > What's "ping PR"? Do you mean pinging the LLVM organization?
> 
> It just means posting a comment on the PR that says "ping", as a reminder for 
> reviewers.
> 
> (But please only do that if you're the patch author, or, in the case of a 
> patch author who's not active any more, if you're volunteering to take over 
> the patch from the patch author and do the work of addressing any remaining 
> review comments.)
> 
> > > What's preventing this PR from being merged right now? All checks seem to 
> > > have passed.
> > 
> > 
> > PRs need to be reviewed before they can be merged.
> 
> Yes, code reviewer bandwidth is the bottleneck for clangd patches at the 
> moment.
> 
> I have some other patches in my review queue which I've been trying to get to 
> (and unfortunately haven't had much time to spend on them), so I'm hoping 
> Kadir can review this one, especially as he has reviewed earlier versions of 
> this and related patches and has more familiarity with the affected code 
> areas.
> 
> But if no one else gets to this, I would like to at some point.

If seems this PR is still stuck.

https://github.com/llvm/llvm-project/pull/78491
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to