[clang] [clang-tools-extra] [clangd] Support parsing comments without ASTContext (PR #78491)

Diogo Behrens via cfe-commits cfe-commits at lists.llvm.org
Sat Oct 12 03:42:46 PDT 2024


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


More information about the cfe-commits mailing list