[PATCH] D59639: [clangd] Print template arguments helper

Ilya Biryukov via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Mar 22 02:50:28 PDT 2019


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





More information about the cfe-commits mailing list