staronj added inline comments.
================
Comment at: clang-tidy/performance/ReturningTypeCheck.cpp:132
@@ +131,3 @@
+ // "constructedType"
+ auto HasTypeSameAsConstructed = hasType(hasCanonicalType(
+ ignoringRefsAndConsts(equalsBoundNode("constructedType"))));
----------------
Prazek wrote:
> I am aware of LLVM style guide, but I think there is exception for the
> matchers - they looks much more like a matchers when they starts with lower
> case. I am not sure if it is not spoken rule, but I don't remember single
> binded matcher that would start with capital case.
See UnnecessaryValueParamCheck.cpp line 33 or LoopConvertCheck.cpp lines 37-47.
They all start with capital letter.
http://reviews.llvm.org/D21303
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
