Re: [PATCH] rs6000: Follow up for signed integer overflow fix

2014-11-20 Thread David Edelsohn
On Thu, Nov 20, 2014 at 1:58 PM, Markus Trippelsdorf wrote: > On 2014.11.20 at 19:44 +0100, Jakub Jelinek wrote: >> On Thu, Nov 20, 2014 at 07:41:43PM +0100, Markus Trippelsdorf wrote: >> > 2014-11-20 Markus Trippelsdorf >> > >> > * config/rs6000/rs6000.c (includes_rldic_lshift_p): Cast 0 t

Re: [PATCH] rs6000: Follow up for signed integer overflow fix

2014-11-20 Thread Markus Trippelsdorf
On 2014.11.20 at 19:44 +0100, Jakub Jelinek wrote: > On Thu, Nov 20, 2014 at 07:41:43PM +0100, Markus Trippelsdorf wrote: > > 2014-11-20 Markus Trippelsdorf > > > > * config/rs6000/rs6000.c (includes_rldic_lshift_p): Cast 0 to unsigned. > > (includes_rldicr_lshift_p): Likewise. > > > >

Re: [PATCH] rs6000: Follow up for signed integer overflow fix

2014-11-20 Thread Jakub Jelinek
On Thu, Nov 20, 2014 at 07:41:43PM +0100, Markus Trippelsdorf wrote: > 2014-11-20 Markus Trippelsdorf > > * config/rs6000/rs6000.c (includes_rldic_lshift_p): Cast 0 to unsigned. > (includes_rldicr_lshift_p): Likewise. > > diff --git a/gcc/config/rs6000/rs6000.c b/gcc/config/rs6000/

[PATCH] rs6000: Follow up for signed integer overflow fix

2014-11-20 Thread Markus Trippelsdorf
On 2014.11.20 at 08:59 -0500, David Edelsohn wrote: > On Thu, Nov 20, 2014 at 8:27 AM, Markus Trippelsdorf > wrote: > > Running the testsuite after bootstrap-ubsan on gcc112 shows several issues. > > See > > https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63426 for the full list. > > > > This patch