<html><head><meta http-equiv="Content-Type" content="text/html; charset=us-ascii"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; line-break: after-white-space;" class=""><br class=""><div><br class=""><blockquote type="cite" class=""><div class="">On Aug 1, 2017, at 4:16 PM, Rafael Avila de Espindola <<a href="mailto:rafael.espindola@gmail.com" class="">rafael.espindola@gmail.com</a>> wrote:</div><br class="Apple-interchange-newline"><div class=""><div class="">George Rimar <<a href="mailto:grimar@accesssoftek.com" class="">grimar@accesssoftek.com</a>> writes:<br class=""><br class=""><blockquote type="cite" class="">I used LLVM algorithms like llvm::find_if, llvm::find as areference. <br class="">They use r-value and I think they do that to support constructions like:<br class="">parallelForEach(std::vector<int>({1, 2, 3}), [](int) { ... });<br class=""><br class="">I could probably use "const R &Range" for supporting above, <br class="">but use of r-value is just consistent.<br class=""></blockquote><br class="">Pete, looks like you added the first one of these (all_of). Do you<br class="">remember why you used r-value instead of const references?<br class=""></div></div></blockquote>Looks like it spun out of feedback from Dave Blaikie, but I don't see a specific suggestion to use r-value, so i honestly can't really remember.</div><div><br class=""></div><div>See <a href="http://lists.llvm.org/pipermail/llvm-commits/Week-of-Mon-20150511/276117.html" class="">http://lists.llvm.org/pipermail/llvm-commits/Week-of-Mon-20150511/276117.html</a> for more details, not that it has the r-value stuff.</div><div><br class=""></div><div>Thanks,</div><div>Pete<br class=""><blockquote type="cite" class=""><div class=""><div class=""><br class="">Thanks,<br class="">Rafael<br class=""></div></div></blockquote></div><br class=""></body></html>