> Hi. > > As seen on ppc64le during compilation of Firefox with LTO, combining inchash > value > with a pointer, enum value and an integer, one can eventually get zero value. > Thus I decided to introduce a new flag that would distinguish between not set > hash value > and a valid and (possibly) zero value. > > I've been running regression tests, ready to install after it finishes? > Martin
> >From 952ca6f6c0f99bcd965825898970453fb413964e Mon Sep 17 00:00:00 2001 > From: marxin <mli...@suse.cz> > Date: Fri, 11 Nov 2016 16:15:20 +0100 > Subject: [PATCH] Add sem_item::m_hash_set (PR ipa/78309) > > gcc/ChangeLog: > > 2016-11-15 Martin Liska <mli...@suse.cz> > > PR ipa/78309 > * ipa-icf.c (void sem_item::set_hash): Update m_hash_set. > (sem_function::get_hash): Make condition based on m_hash_set. > (sem_variable::get_hash): Likewise. > * ipa-icf.h (sem_item::m_hash_set): New property. Yep, zero is definitly valid hash value:0 Patch is OK. We may consider backporting it to release branches. Honza
