[clang-tidy] Do not move parameter if only DeclRefExpr occurs inside of a loop

Summary: This fixes a bug where the performance-unnecessary-value-param check suggests a fix to move the parameter inside of a loop which could be invoked multiple times.

Reviewers: sbenza, aaron.ballman, alexfh

Subscribers: JDevlieghere, cfe-commits

Differential Revision: https://reviews.llvm.org/D27187

git-svn-id: https://llvm.org/svn/llvm-project/clang-tools-extra/trunk@289912 91177308-0d34-0410-b5e6-96231b3b80d8
2 files changed