[PATCH] D22440: Use a faster implementation of maxUIntN.

David Majnemer via llvm-commits llvm-commits at lists.llvm.org
Mon Jul 18 10:10:48 PDT 2016


Should be fixed in r275815.

On Mon, Jul 18, 2016 at 9:55 AM, Ivan Krasin <krasin at google.com> wrote:

> krasin added a subscriber: krasin.
> krasin added a comment.
>
> FYI: this made UBSan bot unhappy:
> http://lab.llvm.org:8011/builders/sanitizer-x86_64-linux-fast/builds/14727/steps/check-llvm%20ubsan/logs/stdio
>
>   [ RUN      ] MathExtras.minIntN
>
> /mnt/b/sanitizer-buildbot3/sanitizer-x86_64-linux-fast/build/llvm/include/llvm/Support/MathExtras.h:340:10:
> runtime error: negation of -9223372036854775808 cannot be represented in
> type 'long'; cast to an unsigned type to negate this value to itself
>       #0 0x4fb9e3 in llvm::minIntN(long)
> /mnt/b/sanitizer-buildbot3/sanitizer-x86_64-linux-fast/build/llvm/include/llvm/Support/MathExtras.h:340:10
>
>
> Repository:
>   rL LLVM
>
> https://reviews.llvm.org/D22440
>
>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20160718/c1145e22/attachment.html>


More information about the llvm-commits mailing list