This is an archive of the discontinued LLVM Phabricator instance.

[clangd] Switch to Dex by default for the static index
ClosedPublic

Authored by kbobyrev on Aug 28 2018, 6:19 AM.

Details

Summary

Dex is now mature enough to be used as the default static index. This patch performs the switch but introduces a hidden flag to allow users fallback to Mem in case something happens.

Diff Detail

Event Timeline

kbobyrev created this revision.Aug 28 2018, 6:19 AM
kbobyrev updated this revision to Diff 162840.Aug 28 2018, 6:19 AM

Run clang-format.

ioeric added inline comments.Aug 28 2018, 6:22 AM
clang-tools-extra/clangd/tool/ClangdMain.cpp
32

I think we should stick to the same option and just flip the default. Introducing yet another option (that is going to be removed) could be confusing.

33

Also add FIXME indicating that the option will be removed soon.

kbobyrev updated this revision to Diff 162842.Aug 28 2018, 6:25 AM
kbobyrev marked 2 inline comments as done.
ioeric accepted this revision.Aug 28 2018, 6:57 AM
ioeric added inline comments.
clang-tools-extra/clangd/tool/ClangdMain.cpp
32

Maybe: FIXME: remove this option when Dex is stable enough.

This revision is now accepted and ready to land.Aug 28 2018, 6:57 AM
kbobyrev updated this revision to Diff 162865.Aug 28 2018, 7:54 AM
kbobyrev marked an inline comment as done.
This revision was automatically updated to reflect the committed changes.