This is an archive of the discontinued LLVM Phabricator instance.

[clangd] Add docs for configuration knobs in clangd
ClosedPublic

Authored by kadircet on Aug 26 2019, 6:53 AM.

Diff Detail

Event Timeline

kadircet created this revision.Aug 26 2019, 6:53 AM
Herald added a project: Restricted Project. · View Herald TranscriptAug 26 2019, 6:53 AM
sammccall accepted this revision.Aug 26 2019, 7:20 AM
sammccall added inline comments.
clang-tools-extra/docs/clangd/Configuration.rst
11

LSP capabilities generally aren't user-servicable and probably shouldn't be covered here.

13

Maybe mention that clangd command-line options should be specified in an editor-specific way, and point back to the getting started page for details

24

Skip the rationale here IMO - just "Specify your driver as a full path or list of globs,..."

33

this example is insecure - include the whole prefix instead

37

(I think we can drop this sentence, it seems obvious)

42

I'd point them at clangd --help rather than the code.

This revision is now accepted and ready to land.Aug 26 2019, 7:20 AM
kadircet updated this revision to Diff 217154.Aug 26 2019, 7:40 AM
kadircet marked 6 inline comments as done.
  • Address comments
This revision was automatically updated to reflect the committed changes.