[llvm] r250427 - Sample Profiles - Adjust integer types. Mostly NFC.

Diego Novillo via llvm-commits llvm-commits at lists.llvm.org
Fri Oct 16 07:03:15 PDT 2015


On Fri, Oct 16, 2015 at 9:59 AM, Aaron Ballman <aaron at aaronballman.com>
wrote:

> -  void addCalledTargetSamples(int LineOffset, unsigned Discriminator,
> > -                              std::string FName, unsigned Num) {
> > +  void addCalledTargetSamples(uint32_t LineOffset, uint32_t
> Discriminator,
> > +                              std::string FName, uint64_t Num) {
> >      assert(LineOffset >= 0);
>
> This change causes a -Wtype-limits warning with the assert. The assert
> can be removed. There's another -Wtype-limits warning on line 223 as
> well.
>

Thanks.  That should be fixed by r250513, which I just committed.


Diego.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20151016/5f982902/attachment.html>


More information about the llvm-commits mailing list