Re: [PATCH] Cleanup of IPA-CP alignment lattices

2015-10-08 Thread Martin Jambor
Hi, On Thu, Oct 08, 2015 at 12:08:15AM +0200, Jan Hubicka wrote: > > > This patch broke Solaris bootstrap in stage 1 with g++ 4.9: > > > > > > /vol/gcc/src/hg/trunk/solaris/gcc/ipa-cp.c: In member function 'bool > > > ipcp_alignment_lattice::meet_with_1(unsigned int, unsigned int)': > > > /vol/g

Re: [PATCH] Cleanup of IPA-CP alignment lattices

2015-10-07 Thread Jan Hubicka
> > This patch broke Solaris bootstrap in stage 1 with g++ 4.9: > > > > /vol/gcc/src/hg/trunk/solaris/gcc/ipa-cp.c: In member function 'bool > > ipcp_alignment_lattice::meet_with_1(unsigned int, unsigned int)': > > /vol/gcc/src/hg/trunk/solaris/gcc/ipa-cp.c:855:56: error: call of > > overloaded

Re: [PATCH] Cleanup of IPA-CP alignment lattices

2015-10-07 Thread Martin Jambor
Hi, On Wed, Oct 07, 2015 at 05:04:13PM +0200, Rainer Orth wrote: > Jan Hubicka writes: > > >> Hi, > >> > >> I have already proposed this patch many months ago but it got > >> forgotten, for a number of reasons. It does not change functionality, > >> it only cleans up the alignment lattices, wh

Re: [PATCH] Cleanup of IPA-CP alignment lattices

2015-10-07 Thread Rainer Orth
Jan Hubicka writes: >> Hi, >> >> I have already proposed this patch many months ago but it got >> forgotten, for a number of reasons. It does not change functionality, >> it only cleans up the alignment lattices, which are currently a bit >> clumsy. >> >> I have refreshed the patch and made su

Re: [PATCH] Cleanup of IPA-CP alignment lattices

2015-10-02 Thread Jan Hubicka
> Hi, > > I have already proposed this patch many months ago but it got > forgotten, for a number of reasons. It does not change functionality, > it only cleans up the alignment lattices, which are currently a bit > clumsy. > > I have refreshed the patch and made sure it still bootstraps and tes

[PATCH] Cleanup of IPA-CP alignment lattices

2015-10-02 Thread Martin Jambor
Hi, I have already proposed this patch many months ago but it got forgotten, for a number of reasons. It does not change functionality, it only cleans up the alignment lattices, which are currently a bit clumsy. I have refreshed the patch and made sure it still bootstraps and tests fine on an x8