[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