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

[Kadir Cetinkaya via Phabricator via cfe-commits]

kadircet requested changes to this revision.
kadircet added a comment.
This revision now requires changes to proceed.

i agree with Sam's concerns here. clangd isn't designed to be consumed as a library, but rather as a binary through LSP. increasing surface are here and letting people build applications on top of clangd internals would create extra maintenance burden that we're not equipped to support.

do you have any specific use case that's pending on this change, or is this a change that might allow things in the future? if you have a specific need it would be better if you can share it directly and we can look for other solutions instead.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D145228