[PATCH][LTO] register char_type_node
Richard Guenther
rguenther@suse.de
Fri Aug 14 16:51:00 GMT 2009
On Fri, 14 Aug 2009, Richard Guenther wrote:
> On Fri, 14 Aug 2009, Diego Novillo wrote:
>
> > On Fri, Aug 14, 2009 at 10:42, Richard Guenther<rguenther@suse.de> wrote:
> >
> > > Yeah. Â I was wondering whether we shouldn't register all types
> > > we preload the streamer cache with early - that would also make
> > > us canonicalize to the integer type variants with the pretty names,
> > > not this <unnamed-signed:8> crap I see everywhere around ...
> >
> > Oh, good idea! Yes, that would help quite a bit.
>
> I am testing the following. The 2nd hunk is the CONST_CAST2 issue - the
> IL that is considered valid by the middle-end depends on alias set
> compatibility of pointed-to types. So we should follow the C frontend
> here assigning const char * and char * the same alias set. That
> fixes the collect2 build.
I have not committed the gimple.c piece as it exposes an existing bug
in that we have two pointer types T1->S1 and T2->S2 with
TYPE_MAIN_VARIANT (S1) == S2
TYPE_CANONICAL (T2) == T1
which makes get_alias_set go in cycles. Hmmmm.
Richard.
More information about the Gcc-patches
mailing list