Re: Document __builtin_*_overflow

2014-11-25 Thread Gerald Pfeifer
Hi Jakub, On Wednesday 2014-11-12 14:13, Jakub Jelinek wrote: This patch mentions __builtin_*_overflow in gcc-5/changes.html. Ok for CVS? I've fallen a bit behind with GCC patches, sorry. What do you think about this follow-up patch on top of yours? Gerald Index: changes.html

Re: Document __builtin_*_overflow

2014-11-25 Thread Jakub Jelinek
On Tue, Nov 25, 2014 at 07:50:02PM +0100, Gerald Pfeifer wrote: Hi Jakub, On Wednesday 2014-11-12 14:13, Jakub Jelinek wrote: This patch mentions __builtin_*_overflow in gcc-5/changes.html. Ok for CVS? I've fallen a bit behind with GCC patches, sorry. What do you think about this

Document __builtin_*_overflow

2014-11-12 Thread Jakub Jelinek
Hi! This patch mentions __builtin_*_overflow in gcc-5/changes.html. Ok for CVS? --- gcc-5/changes.html 27 Oct 2014 18:05:26 - 1.20 +++ gcc-5/changes.html 12 Nov 2014 13:09:01 - @@ -84,7 +84,36 @@ of the standard directive code#include/code and the extension

Re: Document __builtin_*_overflow

2014-11-12 Thread Richard Biener
On Wed, Nov 12, 2014 at 2:13 PM, Jakub Jelinek ja...@redhat.com wrote: Hi! This patch mentions __builtin_*_overflow in gcc-5/changes.html. Ok for CVS? Ok. Thanks, Richard. --- gcc-5/changes.html 27 Oct 2014 18:05:26 - 1.20 +++ gcc-5/changes.html 12 Nov 2014 13:09:01 - @@

Re: Document __builtin_*_overflow

2014-11-12 Thread Tristan Gingold
On 12 Nov 2014, at 14:13, Jakub Jelinek ja...@redhat.com wrote: + size_t sz; + if (__builtin_mul_overflow (x, y, amp;sz) Missing right parenthesis ?

Re: Document __builtin_*_overflow

2014-11-12 Thread Jakub Jelinek
On Wed, Nov 12, 2014 at 03:39:25PM +0100, Tristan Gingold wrote: On 12 Nov 2014, at 14:13, Jakub Jelinek ja...@redhat.com wrote: + size_t sz; + if (__builtin_mul_overflow (x, y, amp;sz) Missing right parenthesis ? Thanks, fixed. Jakub