[PATCH] D90588: [clangd] NFC: Only pass ASTContext and TokenBuffer in getFoldingRanges API

[Kirill Bobyrev via Phabricator via cfe-commits]

kbobyrev added a comment.

In D90588#2368021 <https://reviews.llvm.org/D90588#2368021>, @sammccall wrote:

> This doesn't feel quite right to me - we're going to need to get PP conditional regions, include blocks etc from the ParsedAST (they're not in ASTContext).
> My sense is that we'll need a fairly random subset of ParsedAST, and so ParsedAST is a reasonable abstraction unless it's hard to produce for tests. But it isn't!
>
> What's the motivation for this change?

Aww that comment is regarding `syntax::buildSyntaxTree` weren't you? I thought this was about `getFoldingRanges` API... Sorry.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D90588/new/

https://reviews.llvm.org/D90588