[PATCH] D31821: Remove redundant copy in recurrences

[Wei Mi via Phabricator via llvm-commits]

wmi added inline comments.


================
Comment at: lib/CodeGen/PeepholeOptimizer.cpp:1549-1552
+  if (TargetRegs.count(Reg)) {
+    RCs.push_back(RC);
+    return;
+  }
----------------
For findTargetRecurrence, RCs will at most contain one RC, right? It may be better to remove RCs, change the return value of findTargetRecurrence to bool type, and use it to indicate whether a RC is found. 


https://reviews.llvm.org/D31821