Re: [PATCH] Fix PR55489, memory-hog bug in GCSE

2012-12-04 Thread Paolo Bonzini
Il 04/12/2012 23:30, H.J. Lu ha scritto: > On Tue, Nov 27, 2012 at 8:00 AM, Paolo Bonzini wrote: >> Hi, >> >> this bug triggers in the compilation of QEMU with GCC 4.7.2. It is >> latent on trunk because reg_known_value is completely broken. I'll >> send a separate patch, but this one applies th

Re: [PATCH] Fix PR55489, memory-hog bug in GCSE

2012-12-04 Thread H.J. Lu
On Tue, Nov 27, 2012 at 8:00 AM, Paolo Bonzini wrote: > Hi, > > this bug triggers in the compilation of QEMU with GCC 4.7.2. It is > latent on trunk because reg_known_value is completely broken. I'll > send a separate patch, but this one applies there too. > > The problem arises when you have -f

Re: [PATCH] Fix PR55489, memory-hog bug in GCSE

2012-11-27 Thread Eric Botcazou
> Bootstrap finished on x86_64-pc-linux-gnu, regtest in progress; ok for > 4.7 and trunk if it passes? > > Paolo > > 2012-11-26 Paolo Bonzini > > PR rtl-optimization/55489 > * gcse.c (compute_transp): Precompute a canonical version > of XEXP (x, 0), and pass it to canon_true

Re: [PATCH] Fix PR55489, memory-hog bug in GCSE

2012-11-27 Thread Paolo Bonzini
Il 27/11/2012 17:40, Steven Bosscher ha scritto: > > Note that the bug is present on older branches too, but it became much > > worse sometime between 4.4 and 4.7. > > Probably around the time we (i.e. you and me) taught gcse.c to handle > REG_EQUAL expressions? Hmm, no, that was much earlier. L

Re: [PATCH] Fix PR55489, memory-hog bug in GCSE

2012-11-27 Thread Steven Bosscher
On Tue, Nov 27, 2012 at 5:00 PM, Paolo Bonzini wrote: > Note that the bug is present on older branches too, but it became much > worse sometime between 4.4 and 4.7. Probably around the time we (i.e. you and me) taught gcse.c to handle REG_EQUAL expressions? > 2012-11-26 Paolo Bonzini <> > >

[PATCH] Fix PR55489, memory-hog bug in GCSE

2012-11-27 Thread Paolo Bonzini
Hi, this bug triggers in the compilation of QEMU with GCC 4.7.2. It is latent on trunk because reg_known_value is completely broken. I'll send a separate patch, but this one applies there too. The problem arises when you have -fPIE (or -fPIC) and a huge function with a lot of references to glob