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

Richard Smith via llvm-commits llvm-commits at lists.llvm.org
Thu Nov 19 19:27:06 PST 2015


See the patch I posted on the other thread for this issue -- that should
avoid the UB and the division in all cases.
On Nov 19, 2015 5:55 PM, "David Li via llvm-commits" <
llvm-commits at lists.llvm.org> wrote:

> davidxl accepted this revision.
> davidxl added a comment.
> This revision is now accepted and ready to land.
>
> If you have time, finding a way to speed up the method will be welcome
> (such as the fast path suggestion).
>
>
> http://reviews.llvm.org/D14845
>
>
>
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at lists.llvm.org
> http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20151119/9720b405/attachment.html>


More information about the llvm-commits mailing list