[PATCH] D145228: [clangd] Add clangd headers to install targets

Tue Mar 7 08:01:43 PST 2023

ivanmurashko added a comment.

In D145228#4174829 <https://reviews.llvm.org/D145228#4174829>, @sammccall wrote:

> I think this is a bit abstract though. Concretely, what API do you need here? e.g. which headers do you want to include, to what end?

If we consider the bare minimum with the only goal to build outside LLVM source tree then we don’t need to copy all internal headers.  At the case the D145302 <https://reviews.llvm.org/D145302> can be modified and we can end up with the only one header that is required to copy:  clang-tools-extra/clangd/tool/ClangdMain.h.

For further customization one might require additional headers to be installed. That can be done once it’s required or via one diff that install all possible headers.

What do you think about it?

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D145228