Re: [PATCH] Add sem_item::m_hash_set (PR ipa/78309) (v2)

2016-11-24 Thread Jan Hubicka
> On 11/21/2016 04:50 PM, Jan Hubicka wrote: > > OK, > > thanks! > > Honza > > Hi. > > Patch to trunk is already installed. Equal patch can be installed to gcc-6 > branch, > however gcc-5 branch needs more hunks to be adjusted. I did so, both patches > survive > regression tests and the patch f

Re: [PATCH] Add sem_item::m_hash_set (PR ipa/78309) (v2)

2016-11-23 Thread Martin Liška
On 11/21/2016 04:50 PM, Jan Hubicka wrote: > OK, > thanks! > Honza Hi. Patch to trunk is already installed. Equal patch can be installed to gcc-6 branch, however gcc-5 branch needs more hunks to be adjusted. I did so, both patches survive regression tests and the patch for gcc-5 provides equal

Re: [PATCH] Add sem_item::m_hash_set (PR ipa/78309) (v2)

2016-11-21 Thread Jan Hubicka
> On 11/15/2016 05:46 PM, Jan Hubicka wrote: > > Yep, zero is definitly valid hash value:0 > > > > Patch is OK. We may consider backporting it to release branches. > > Honza > > Thanks, sending v2 as I found an error in the previous version. > Changes from last version: > - comments for ctors are

Re: [PATCH] Add sem_item::m_hash_set (PR ipa/78309) (v2)

2016-11-16 Thread Martin Liška
On 11/15/2016 05:46 PM, Jan Hubicka wrote: > Yep, zero is definitly valid hash value:0 > > Patch is OK. We may consider backporting it to release branches. > Honza Thanks, sending v2 as I found an error in the previous version. Changes from last version: - comments for ctors are just in header fi