<div dir="rtl"><div dir="ltr"><a href="http://reviews.llvm.org/D4018">http://reviews.llvm.org/D4018</a><br></div><div dir="ltr"><br></div><div dir="ltr"><span style="color:rgb(0,0,0);font-family:'Helvetica Neue',Helvetica,Arial,sans-serif;font-size:13px;line-height:18.850000381469727px">Updated upatch to ToT, fixed comments and two warnings.</span><br>

</div><div dir="ltr"><span style="color:rgb(0,0,0);font-family:'Helvetica Neue',Helvetica,Arial,sans-serif;font-size:13px;line-height:18.850000381469727px"><br></span></div></div><div class="gmail_extra"><br><br>
<div class="gmail_quote">
<div dir="ltr">2014-06-04 0:11 GMT+03:00 Rafael Espíndola <span dir="ltr"><<a href="mailto:rafael.espindola@gmail.com" target="_blank">rafael.espindola@gmail.com</a>></span>:</div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

+  /// count -<br>
<br>
While at it, please drop the name from the comment.<br>
<br>
I think the patch is OK. Richard, do you know what is the reason the<br>
stl uses size_type?<br>
<div><div class="h5"><br>
<br>
On 23 December 2013 06:51, Yaron Keren <<a href="mailto:yaron.keren@gmail.com">yaron.keren@gmail.com</a>> wrote:<br>
> The count() function for STL datatypes returns unsigned, even where it's<br>
> only 1/0 result like std::set. Some of the LLVM ADT already return unsigned<br>
> count(), while others still return bool count().<br>
><br>
> In continuation to r197879, this patch modifies DenseMap, DenseSet,<br>
> ScopedHashTable, ValueMap:: count() to return unsigned instead of bool, 1<br>
> instead of true and 0 instead of false.<br>
><br>
> Yaron<br>
><br>
><br>
</div></div>> _______________________________________________<br>
> cfe-commits mailing list<br>
> <a href="mailto:cfe-commits@cs.uiuc.edu">cfe-commits@cs.uiuc.edu</a><br>
> <a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits</a><br>
><br>
</blockquote></div><br></div>