[PATCH] D57057: [clangd] Log clang-tidy configuration, NFC

2019-01-29 Thread Haojian Wu via Phabricator via cfe-commits
hokein added a comment. In D57057#1375249 , @sammccall wrote: > This turns out to be excessively spammy in unit tests, can we change this to > dlog()? Sure, done in rL352485 . Repository: rL LLVM CHANGES SINCE LAS

[PATCH] D57057: [clangd] Log clang-tidy configuration, NFC

2019-01-29 Thread Sam McCall via Phabricator via cfe-commits
sammccall added a comment. This turns out to be excessively spammy in unit tests, can we change this to dlog()? Repository: rL LLVM CHANGES SINCE LAST ACTION https://reviews.llvm.org/D57057/new/ https://reviews.llvm.org/D57057 ___ cfe-commits

[PATCH] D57057: [clangd] Log clang-tidy configuration, NFC

2019-01-25 Thread Haojian Wu via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL352184: [clangd] Log clang-tidy configuration, NFC (authored by hokein, committed by ). Herald added a subscriber: llvm-commits. Repository: rL LLVM CHANGES SINCE LAST ACTION https://reviews.llvm.org

[PATCH] D57057: [clangd] Log clang-tidy configuration, NFC

2019-01-24 Thread Jan Korous via Phabricator via cfe-commits
jkorous accepted this revision. jkorous added a comment. This revision is now accepted and ready to land. LGTM Repository: rCTE Clang Tools Extra CHANGES SINCE LAST ACTION https://reviews.llvm.org/D57057/new/ https://reviews.llvm.org/D57057 __

[PATCH] D57057: [clangd] Log clang-tidy configuration, NFC

2019-01-22 Thread Haojian Wu via Phabricator via cfe-commits
hokein created this revision. hokein added a reviewer: sammccall. Herald added subscribers: kadircet, arphaman, jkorous, MaskRay, ioeric, ilya-biryukov. This is used for debugging purpose. Repository: rCTE Clang Tools Extra https://reviews.llvm.org/D57057 Files: clangd/ClangdUnit.cpp In