Re: [PATCH v2] Fix signed integer overflow in gcc/data-streamer.c

2014-09-30 Thread Diego Novillo
On Tue, Sep 30, 2014 at 3:09 AM, Markus Trippelsdorf wrote: > On 2014.09.28 at 14:57 +0200, Markus Trippelsdorf wrote: >> On 2014.09.28 at 14:36 +0200, Steven Bosscher wrote: >> > >> > Can you use HOST_WIDE_INT_1U for this? >> >> Sure. Thanks for the suggestion. >> (Fix now resembles similar idiom

Re: [PATCH v2] Fix signed integer overflow in gcc/data-streamer.c

2014-09-30 Thread Markus Trippelsdorf
On 2014.09.28 at 14:57 +0200, Markus Trippelsdorf wrote: > On 2014.09.28 at 14:36 +0200, Steven Bosscher wrote: > > > > Can you use HOST_WIDE_INT_1U for this? > > Sure. Thanks for the suggestion. > (Fix now resembles similar idiom in data-streamer-in.c) I checked in the fix as obvious. -- Mar

[PATCH v2] Fix signed integer overflow in gcc/data-streamer.c

2014-09-28 Thread Markus Trippelsdorf
On 2014.09.28 at 14:36 +0200, Steven Bosscher wrote: > > Can you use HOST_WIDE_INT_1U for this? Sure. Thanks for the suggestion. (Fix now resembles similar idiom in data-streamer-in.c) 2014-09-28 Markus Trippelsdorf * data-streamer.c (bp_unpack_var_len_int): Avoid signed int