[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