[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