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

Nathan Ridge via cfe-commits cfe-commits at lists.llvm.org
Mon Jun 10 22:32:34 PDT 2024


HighCommander4 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.

https://github.com/llvm/llvm-project/pull/78491


More information about the cfe-commits mailing list