Page MenuHomePhabricator

[clangd] Use Dex for dynamic index by default.
ClosedPublic

Authored by ioeric on Feb 7 2019, 1:17 AM.

Details

Summary

Memory usage for a sample TU:

Without Dex: 17.9M
With    Dex: 24.4M

The memory increase is considerable but seems tolerable.

Diff Detail

Event Timeline

ioeric created this revision.Feb 7 2019, 1:17 AM
Herald added a project: Restricted Project. · View Herald TranscriptFeb 7 2019, 1:17 AM
sammccall accepted this revision.Feb 7 2019, 7:24 AM

(Just flipping the flag seems fine too)

This revision is now accepted and ready to land.Feb 7 2019, 7:24 AM
ioeric updated this revision to Diff 185762.Feb 7 2019, 7:33 AM
  • Keep the option for now.
This revision was automatically updated to reflect the committed changes.