[clang-tools-extra] [llvm] [Clangd] Add AST search capabilities from clang-query (PR #156090)

[Nathan Ridge via llvm-commits]

Fabian =?utf-8?q?Keßler?= <fabian_kessler at gmx.de>,
Fabian =?utf-8?q?Keßler?= <fabian_kessler at gmx.de>,
Fabian =?utf-8?q?Keßler?= <fabian_kessler at gmx.de>,
Fabian =?utf-8?q?Keßler?= <fabian_kessler at gmx.de>
Message-ID:
In-Reply-To: <llvm.org/llvm/llvm-project/pull/156090 at github.com>


HighCommander4 wrote:

> My plan was just to move the existing "ast view" treeview widget to a new menu, add a text bar above, and allow it to show multiple matches for a first prototype.

I'm wondering, what is the use case for showing the entire AST subtree (and indeed, the protocol sending an entire `ASTNode`) for each match?

Might it be sufficient to just send a text file range (much like ordinary search), and then if the user wants to see the AST for a particular result they can select the range and invoke "Show AST"?

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