[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