Re: [PATCH, PR 55253] Propagate aggs_contain_variable flag in aggregater IPA-CP

2012-11-14 Thread Martin Jambor
Hi, On Tue, Nov 13, 2012 at 01:08:56PM -0800, Ian Lance Taylor wrote: > On Tue, Nov 13, 2012 at 9:03 AM, Martin Jambor wrote: > > > Index: src/gcc/ipa-cp.c > > === > > --- src.orig/gcc/ipa-cp.c > > +++ src/gcc/ipa-cp.c > > @@ -1276,

Re: [PATCH, PR 55253] Propagate aggs_contain_variable flag in aggregater IPA-CP

2012-11-13 Thread Ian Lance Taylor
On Tue, Nov 13, 2012 at 9:03 AM, Martin Jambor wrote: > Index: src/gcc/ipa-cp.c > === > --- src.orig/gcc/ipa-cp.c > +++ src/gcc/ipa-cp.c > @@ -1276,6 +1276,8 @@ merge_aggregate_lattices (struct cgraph_ > return true; >if (sr

Re: [PATCH, PR 55253] Propagate aggs_contain_variable flag in aggregater IPA-CP

2012-11-13 Thread Jan Hubicka
> Hi, > > somehow the propagation of aggs_contain_variable in > merge_aggregate_lattices got lost when I was shuffling the code > around, leading to miscompilations. This patch puts it back in. > > Bootstrapped and tested on x86_64-linux, OK for trunk? > > Thanks, > > Martin > > > 2012-11-13

[PATCH, PR 55253] Propagate aggs_contain_variable flag in aggregater IPA-CP

2012-11-13 Thread Martin Jambor
Hi, somehow the propagation of aggs_contain_variable in merge_aggregate_lattices got lost when I was shuffling the code around, leading to miscompilations. This patch puts it back in. Bootstrapped and tested on x86_64-linux, OK for trunk? Thanks, Martin 2012-11-13 Martin Jambor P