This is an archive of the discontinued LLVM Phabricator instance.

[clangd] Log directory when a CDB is loaded
ClosedPublic

Authored by kadircet on Jan 29 2020, 7:22 AM.

Diff Detail

Event Timeline

kadircet created this revision.Jan 29 2020, 7:22 AM
Herald added a project: Restricted Project. · View Herald TranscriptJan 29 2020, 7:22 AM
sammccall accepted this revision.Jan 29 2020, 7:42 AM
sammccall added inline comments.
clang-tools-extra/clangd/GlobalCompilationDatabase.cpp
122

consider "Loaded compile commands..." or "Loading compile commands..." or "Loaded compilation database...".
People who are only dimly aware of the existence of compile_commands.json need a bit more context here.

122

this is significant and rare, probably OK to log rather than vlog

This revision is now accepted and ready to land.Jan 29 2020, 7:42 AM

Unit tests: pass. 62284 tests passed, 0 failed and 831 were skipped.

clang-tidy: pass.

clang-format: pass.

Build artifacts: diff.json, clang-tidy.txt, clang-format.patch, CMakeCache.txt, console-log.txt, test-results.xml

Pre-merge checks is in beta. Report issue. Please join beta or enable it for your project.

kadircet updated this revision to Diff 241388.Jan 30 2020, 3:14 AM
kadircet marked 2 inline comments as done.
  • Address comments
This revision was automatically updated to reflect the committed changes.