This is an archive of the discontinued LLVM Phabricator instance.

[clangd][Background] Make index validation logs verbose
ClosedPublic

Authored by kadircet on Jan 20 2022, 8:26 AM.

Details

Summary

These errors are non-harmful and should be transient. They either
imply:

  • compilation database returned stale results for TUs and it'll be fixed once it's updated to match project state.
  • a TUs dependencies has changed and some headers no longer exist. this should be fixed with the next indexing cycle.

In either case the user will have some stale symbols in their index until clangd
restarts and the underlying issue is resolved. On the downside these logs are
confusing users when there's another issue.

Diff Detail

Event Timeline

kadircet created this revision.Jan 20 2022, 8:26 AM
kadircet requested review of this revision.Jan 20 2022, 8:26 AM
sammccall accepted this revision.Jan 20 2022, 8:32 AM
This revision is now accepted and ready to land.Jan 20 2022, 8:32 AM
This revision was automatically updated to reflect the committed changes.