[PATCH] D35225: [clang-tidy] add regression test to performance-unnecessary-value-param

2017-07-12 Thread Chih-Hung Hsieh via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL307810: [clang-tidy] add regression test to performance-unnecessary-value-param (authored by chh). Changed prior to commit: https://reviews.llvm.org/D35225?vs=106064=106241#toc Repository: rL LLVM

[PATCH] D35225: [clang-tidy] add regression test to performance-unnecessary-value-param

2017-07-12 Thread Alexander Kornienko via Phabricator via cfe-commits
alexfh accepted this revision. alexfh added a comment. This revision is now accepted and ready to land. LG. Thanks! https://reviews.llvm.org/D35225 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

[PATCH] D35225: [clang-tidy] add regression test to performance-unnecessary-value-param

2017-07-11 Thread Chih-Hung Hsieh via Phabricator via cfe-commits
chh updated this revision to Diff 106064. chh marked 2 inline comments as done. https://reviews.llvm.org/D35225 Files: test/clang-tidy/Inputs/performance-unnecessary-value-param/header-fixed.h test/clang-tidy/Inputs/performance-unnecessary-value-param/header.h

[PATCH] D35225: [clang-tidy] add regression test to performance-unnecessary-value-param

2017-07-11 Thread Alexander Kornienko via Phabricator via cfe-commits
alexfh requested changes to this revision. alexfh added inline comments. This revision now requires changes to proceed. Comment at: test/clang-tidy/performance-unnecessary-value-param.cpp:1 -// RUN: %check_clang_tidy %s performance-unnecessary-value-param %t +// RUN: cp

[PATCH] D35225: [clang-tidy] add regression test to performance-unnecessary-value-param

2017-07-10 Thread Chih-Hung Hsieh via Phabricator via cfe-commits
chh added a comment. These tests should be added after https://bugs.llvm.org/show_bug.cgi?id=33734 is fixed. https://reviews.llvm.org/D35225 ___ cfe-commits mailing list cfe-commits@lists.llvm.org