ilya-biryukov added inline comments.

================
Comment at: lib/Index/IndexingContext.cpp:51
+
 bool IndexingContext::handleDecl(const Decl *D,
                                  SymbolRoleSet Roles,
----------------
Do we call `handleDecl` for template parameters now too?


================
Comment at: unittests/Index/IndexTests.cpp:141
+    template <typename T, int I> struct Foo {
+      T t = I;
+    };
----------------
could we add a template template parameter into the test too?


Repository:
  rC Clang

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D58293/new/

https://reviews.llvm.org/D58293



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to