Kessoufi marked an inline comment as done.
Kessoufi added a comment.
Done
http://reviews.llvm.org/D19194
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Kessoufi updated this revision to Diff 54524.
Kessoufi added a comment.
removed the redundant matcher
http://reviews.llvm.org/D19194
Files:
clang-tidy/modernize/PassByValueCheck.cpp
Index: clang-tidy/modernize/PassByValueCheck.cpp
=
Kessoufi updated this revision to Diff 54762.
Kessoufi added a comment.
- my previous fix was applied on the wrong location and accidentally worked.
now applied on the right location
- previous tests still pass
- added a specific test to highlight my issue
http://reviews.llvm.org/D19194
Files
