This is an archive of the discontinued LLVM Phabricator instance.

gn build: Add some build files for clangd
ClosedPublic

Authored by thakis on Mar 27 2019, 1:29 PM.

Details

Summary

Enough to build the clangd binaries, but this is still missing build
files for:

  • fuzzer
  • indexer
  • index/dex/dexp
  • benchmarks
  • xpc

Diff Detail

Repository
rL LLVM

Event Timeline

thakis created this revision.Mar 27 2019, 1:29 PM
Herald added a project: Restricted Project. · View Herald TranscriptMar 27 2019, 1:29 PM
mbonadei accepted this revision.Mar 28 2019, 8:28 AM

Sorry for the delay, LGTM % 2 small comments.

llvm/utils/gn/secondary/clang-tools-extra/clang-apply-replacements/BUILD.gn
2 ↗(On Diff #192509)

Oh, I forgot to add this! Thanks!

llvm/utils/gn/secondary/clang-tools-extra/clangd/BUILD.gn
32 ↗(On Diff #192509)

Missing //clang/lib/Tooling ?

llvm/utils/gn/secondary/clang-tools-extra/clangd/refactor/tweaks/BUILD.gn
7 ↗(On Diff #192509)

Missing "//llvm/lib/Support"?

Probably it is ok, since it is a source_set, but am I following the 1:1 matching rule.

llvm/utils/gn/secondary/clang-tools-extra/clangd/tool/BUILD.gn
9 ↗(On Diff #192509)

+1, if that is not possible this target should be in clangd/BUILD.gn (it is more natural).

This revision is now accepted and ready to land.Mar 28 2019, 8:28 AM
thakis marked 2 inline comments as done.Mar 28 2019, 9:46 AM

Thanks! Landing with comments addressed.

This revision was automatically updated to reflect the committed changes.