This is an archive of the discontinued LLVM Phabricator instance.

[clangd] Give an example for global-symbol-builder usage
ClosedPublic

Authored by kbobyrev on Jul 25 2018, 4:40 AM.

Details

Summary

global-symbol-builder help message mentions -executor=<string> option, but doesn't give any example of what the value could be

Assuming the most popular use case to be building the whole project index, help message should probably give an example of such usage.

Diff Detail

Repository
rL LLVM

Event Timeline

kbobyrev created this revision.Jul 25 2018, 4:40 AM
ioeric accepted this revision.Jul 25 2018, 5:11 AM
ioeric added inline comments.
clang-tools-extra/clangd/global-symbol-builder/GlobalSymbolBuilderMain.cpp
153 ↗(On Diff #157227)

why replace char * with auto? I'd keep char * as it's clearer and short enough.

158 ↗(On Diff #157227)

Maybe also give an example for running on one file?

This revision is now accepted and ready to land.Jul 25 2018, 5:11 AM
kbobyrev updated this revision to Diff 157263.Jul 25 2018, 7:31 AM
kbobyrev marked 2 inline comments as done.

Addressed two comments, added an example of running the tool over a number of source files without providing compile_commands.json; added a note about collecting files from headers only.

This revision was automatically updated to reflect the committed changes.