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

Repository
rL LLVM

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 ↗(On Diff #162838)

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 ↗(On Diff #162838)

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 ↗(On Diff #162842)

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.