[PATCH] D14845: [Support] Fix SaturatingMultiply<T>() to never cause overflow, Re-enable Unit Tests

Xinliang David Li via llvm-commits llvm-commits at lists.llvm.org
Thu Nov 19 17:54:15 PST 2015


ok makes sense.

On Thu, Nov 19, 2015 at 5:36 PM, Nathan Slingerland <slingn at gmail.com>
wrote:

> slingn added a comment.
>
> https://llvm.org/bugs/show_bug.cgi?id=25580
>
> Apparently not a bug with UBSan - there was a valid problem with signed
> integer overflow due to up-conversion of integer types with the previous
> implementation.
>
>
> http://reviews.llvm.org/D14845
>
>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20151119/32a6a617/attachment.html>


More information about the llvm-commits mailing list