[PATCH] D41454: [clangd] Add ClangdUnit diagnostics tests using annotated code.

2018-01-25 Thread Sam McCall via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rCTE323448: [clangd] Add ClangdUnit diagnostics tests using annotated code. (authored by sammccall, committed by ). Changed prior to commit: https://reviews.llvm.org/D41454?vs=127843&id=131471#toc Reposi

[PATCH] D41454: [clangd] Add ClangdUnit diagnostics tests using annotated code.

2018-01-24 Thread Ilya Biryukov via Phabricator via cfe-commits
ilya-biryukov accepted this revision. ilya-biryukov added a comment. This revision is now accepted and ready to land. Herald added subscribers: hintonda, ioeric, jkorous-apple. LGTM. Repository: rCTE Clang Tools Extra https://reviews.llvm.org/D41454

[PATCH] D41454: [clangd] Add ClangdUnit diagnostics tests using annotated code.

2018-01-12 Thread Sam McCall via Phabricator via cfe-commits
sammccall added a comment. ping... just realized this didn't go in Repository: rCTE Clang Tools Extra https://reviews.llvm.org/D41454 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commi

[PATCH] D41454: [clangd] Add ClangdUnit diagnostics tests using annotated code.

2017-12-21 Thread Sam McCall via Phabricator via cfe-commits
sammccall updated this revision to Diff 127843. sammccall added a comment. Tighten preprocessor test Repository: rCTE Clang Tools Extra https://reviews.llvm.org/D41454 Files: clangd/Protocol.cpp clangd/Protocol.h test/clangd/diagnostics-preamble.test unittests/clangd/CMakeLists.txt

[PATCH] D41454: [clangd] Add ClangdUnit diagnostics tests using annotated code.

2017-12-20 Thread Sam McCall via Phabricator via cfe-commits
sammccall created this revision. sammccall added a reviewer: ilya-biryukov. Herald added subscribers: cfe-commits, mgorny, klimek. This adds checks that our diagnostics emit correct ranges in a bunch of cases, as promised in https://reviews.llvm.org/D41118. The diagnostics-preamble test is also c