[PATCH] D55770: [clangd] BackgroundIndex rebuilds symbol index periodically.

David Blaikie via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Dec 17 10:56:08 PST 2018


dblaikie added a comment.

(from the peanut gallery): Usually what I'd expect here is "reindexing will occur X milliseconds after a write, covering anything written up until that point" - so it doesn't reindex unnecessarily if the files are unmodified/sitting idle, and it doesn't thrash reindexing for multiple changes in rapid succession (such as hitting the "save all" button in some text editor - or having made a few other quick changes one after another maybe), but still provides reasonable up-to-date data for the user.

(but I don't know anything about clangd & maybe what I'm saying makes no sense/isn't relevant)


Repository:
  rCTE Clang Tools Extra

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D55770/new/

https://reviews.llvm.org/D55770





More information about the cfe-commits mailing list