ilya-biryukov added inline comments.
================ Comment at: clang-tools-extra/unittests/clangd/ASTUtilsTests.cpp:1 +#include "AST.h" +#include "Annotations.h" ---------------- NIT: add a licence header ================ Comment at: clang-tools-extra/unittests/clangd/CMakeLists.txt:13 Annotations.cpp + ASTUtilsTests.cpp BackgroundIndexTests.cpp ---------------- NIT: maybe call it `PrintASTTests`? To avoid creating a dump of test for all the stuff we are too lazy to classify properly Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D59639/new/ https://reviews.llvm.org/D59639 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
