On Tue, Jun 3, 2008 at 5:28 PM, Michael Matz <[EMAIL PROTECTED]> wrote: > Hi, > > On Tue, 3 Jun 2008, Diego Novillo wrote: > >> On Tue, Jun 3, 2008 at 04:51, Richard Guenther >> <[EMAIL PROTECTED]> wrote: >> >> > You may want to read >> > >> > http://gcc.gnu.org/ml/gcc-patches/2008-03/msg00349.html >> >> Thanks. I think I will try to incorporate this in LTO so we won't >> even need to deal with these codes in the streamer. Michael, are you >> OK with this or were you going to work on this for mainline? > > I'm fine with that. Note, though, that incorporating this potentially > breaks quite some things. In particular debugging and thunks. Due to the > hack for copying C++ classes also that will be broken. I didn't get much > feedback about the thing in general so I stopped for the time being, but > at least some of the changes would also be usable for mainline. > > It adds a uid2type map on top of Richis uid2decl map, which I think is > also a nice thing to have in general. But it has the same garbage > collection implications as Richis uid2decl map, in that it will break > bootstrap in mysterious ways, when something is collected between two > aliasing passes :-/
Note that (bootstrap problems) only occur if the accompaning change of turning the referenced_vars hash into a bitmap. It may be that recent cleanups in aliasing fixed or hid that problem though. Richard.
