<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Fri, Oct 16, 2015 at 9:59 AM, Aaron Ballman <span dir="ltr"><<a href="mailto:aaron@aaronballman.com" target="_blank" class="cremed">aaron@aaronballman.com</a>></span> wrote:</div><div class="gmail_quote"><br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5">
> -  void addCalledTargetSamples(int LineOffset, unsigned Discriminator,<br>
> -                              std::string FName, unsigned Num) {<br>
> +  void addCalledTargetSamples(uint32_t LineOffset, uint32_t Discriminator,<br>
> +                              std::string FName, uint64_t Num) {<br>
>      assert(LineOffset >= 0);<br>
<br>
</div></div>This change causes a -Wtype-limits warning with the assert. The assert<br>
can be removed. There's another -Wtype-limits warning on line 223 as<br>
well.<br></blockquote><div><br></div><div>Thanks.  That should be fixed by r250513, which I just committed.</div><div><br></div><div><br></div><div>Diego.</div></div></div></div>