This is an archive of the discontinued LLVM Phabricator instance.

[clangd] Update the out-of-date yaml-symbol-file flag in clangd.
ClosedPublic

Authored by hokein on Oct 8 2018, 2:39 AM.

Diff Detail

Repository
rL LLVM

Event Timeline

hokein created this revision.Oct 8 2018, 2:39 AM
sammccall accepted this revision.Oct 8 2018, 3:30 AM

Hmm, I wonder if we should remove "experimental" text and the hidden bit. Or wait to see if we want to move to automatic index entirely?
May want to mention "must have been created by a compatible clangd-indexer".

This revision is now accepted and ready to land.Oct 8 2018, 3:30 AM
hokein updated this revision to Diff 168639.Oct 8 2018, 3:44 AM

address comment.

hokein added a comment.Oct 8 2018, 3:44 AM

Hmm, I wonder if we should remove "experimental" text and the hidden bit. Or wait to see if we want to move to automatic index entirely?

Our plan seems unclear at the moment, so I would suggest not doing it now until we sort them out.

May want to mention "must have been created by a compatible clangd-indexer".

Done.

This revision was automatically updated to reflect the committed changes.