[PATCH] D23860: [Sema][Comments] Add support for TypeAliasTemplate

Dmitri Gribenko via cfe-commits cfe-commits at lists.llvm.org
Thu Aug 25 03:03:02 PDT 2016


gribozavr accepted this revision.
gribozavr added a comment.
This revision is now accepted and ready to land.

LGTM with two more tests.


================
Comment at: test/Sema/warn-documentation.cpp:433
@@ +432,3 @@
+template<typename T>
+using test_function_like_usingA = foo::function_wrapper<int (T aaa, int ccc)>;
+
----------------
For consistency with the rest of the file, could you use '11' instead of 'A'?

Could you also add these two tests:

```
/// <please-add-some-comment>
template<typename T>
using test_function_like_using10 = int (*)(T aaa, int ccc);

/// <please-add-some-comment>
template<typename T>
using test_function_like_using12 = foo::function_wrapper<int (T aaa, int ccc)> *;
```


https://reviews.llvm.org/D23860





More information about the cfe-commits mailing list