<div dir="ltr">Benjamin and I were just discussing a similar piece of code to this in a review<br><br>I think we were settling on something like "any_of(R, is_equal<>(E))" but I'm OK adding something like "contains" or "is_contained" (contains seems like the right name, but might be too often used elsewhere to be convenient?)<br><br>Benjamin - any thoughts/preferences here?</div><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Jan 11, 2016 at 5:26 AM, Alexander Droste via llvm-commits <span dir="ltr"><<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Alexander_Droste updated the summary for this revision.<br>
Alexander_Droste added a reviewer: dblaikie.<br>
Alexander_Droste updated this revision to Diff 44471.<br>
Alexander_Droste added a comment.<br>
<br>
- changed name to `is_contained()` because `contains()` caused name conflicts<br>
- adapted the style of other functions in `STLExtras.h`<br>
<br>
- @David: I hope its ok that I added you as a reviewer. Googling for 'stlextras llvm reviewer' you<br>
<br>
were the first author that appeared.<br>
<span class=""><br>
<br>
<a href="http://reviews.llvm.org/D16053" rel="noreferrer" target="_blank">http://reviews.llvm.org/D16053</a><br>
<br>
Files:<br>
  include/llvm/ADT/STLExtras.h<br>
<br>
Index: include/llvm/ADT/STLExtras.h<br>
===================================================================<br>
--- include/llvm/ADT/STLExtras.h<br>
+++ include/llvm/ADT/STLExtras.h<br>
@@ -386,6 +386,13 @@<br>
   return std::find(Range.begin(), Range.end(), val);<br>
 }<br>
<br>
</span>+/// Wrapper function around std::find to detect if an element exists<br>
+/// in a container.<br>
+template <typename R, typename E><br>
+bool is_contained(R && Range, const E &Element) {<br>
+  return std::find(Range.begin(), Range.end(), Element) != Range.end();<br>
<div class="HOEnZb"><div class="h5">+}<br>
+<br>
 //===----------------------------------------------------------------------===//<br>
 //     Extra additions to <memory><br>
 //===----------------------------------------------------------------------===//<br>
<br>
<br>
</div></div><br>_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@lists.llvm.org">llvm-commits@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits</a><br>
<br></blockquote></div><br></div>