[lto] Refactor streamer (1/N) (issue4809083)

Michael Matz matz@suse.de
Fri Aug 26 12:50:00 GMT 2011


Hi,

On Fri, 26 Aug 2011, Richard Guenther wrote:

> >> I am going to be sending the renaming patch later today or tomorrow. 
> >> In principle, the things I want to abstract are those that are 
> >> forcing me to include lto-streamer.h from 
> >> {tree,gimple,data}-streamer.*. I will know better when I merge this 
> >> into the pph branch, though.
> >
> > Yeah, I think we discussed this already and agreed on that this is a 
> > sensible plan.
> 
> This patch caused http://gcc.gnu.org/bugzilla/show_bug.cgi?id=50165 it 
> seems that LTO string hashing is seriously broken now.

Once regstrap passes on x86_64-linux I'm checking this in as obvious.


Ciao,
Michael.
-- 
	PR lto/50165
	* lto-streamer-in.c (canon_file_name): Initialize new_slot->len.

Index: lto-streamer-in.c
===================================================================
--- lto-streamer-in.c	(revision 178040)
+++ lto-streamer-in.c	(working copy)
@@ -113,6 +113,7 @@ canon_file_name (const char *string)
       new_slot = XCNEW (struct string_slot);
       strcpy (saved_string, string);
       new_slot->s = saved_string;
+      new_slot->len = len;
       *slot = new_slot;
       return saved_string;
     }



More information about the Gcc-patches mailing list