[PATCH, PR55315] Don't assume a nonzero address plus a const is a nonzero address

2012-11-17 Thread Tom de Vries
Richard, This patch fixes PR 55315. When compiling for the mips target with -O2, function f is folded to 0, while the address of data is not known compile-time: ... int data[4096]; int f (void) { return (((unsigned int) &data[0]) == 0xdeadbea0U); } What happens is that expand turns the c

Re: [PATCH, PR55315] Don't assume a nonzero address plus a const is a nonzero address

2012-11-18 Thread Richard Sandiford
Tom de Vries writes: > 2012-11-17 Tom de Vries > > PR rtl-optimization/55315 > > * rtlanal.c (nonzero_address_p): Don't assume a nonzero address plus a > const is a nonzero address. > > * gcc.target/mips/pr55315.c: New test. OK, thanks. Richard

Re: [PATCH, PR55315] Don't assume a nonzero address plus a const is a nonzero address

2012-11-19 Thread Jakub Jelinek
On Sun, Nov 18, 2012 at 10:50:53AM +, Richard Sandiford wrote: > Tom de Vries writes: > > 2012-11-17 Tom de Vries > > > > PR rtl-optimization/55315 > > > > * rtlanal.c (nonzero_address_p): Don't assume a nonzero address plus a > > const is a nonzero address. > > > > * gcc.ta

Re: [PATCH, PR55315] Don't assume a nonzero address plus a const is a nonzero address

2012-11-19 Thread Tom de Vries
On 19/11/12 14:00, Jakub Jelinek wrote: > On Sun, Nov 18, 2012 at 10:50:53AM +, Richard Sandiford wrote: >> Tom de Vries writes: >>> 2012-11-17 Tom de Vries >>> >>> PR rtl-optimization/55315 >>> >>> * rtlanal.c (nonzero_address_p): Don't assume a nonzero address plus a >>> const