This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]
Other format:	[Raw text]

Re: [PATCH] Fix PRs 33816, 32921 (again, different)

From: kenner at vlsi1 dot ultra dot nyu dot edu (Richard Kenner)
To: rguenther at suse dot de
Cc: dnovillo at google dot com, gcc-patches at gcc dot gnu dot org
Date: Fri, 19 Oct 2007 17:43:44 EDT
Subject: Re: [PATCH] Fix PRs 33816, 32921 (again, different)
References: <Pine.LNX.4.64.0710191628280.4086@zhemvz.fhfr.qr> <b798aad50710190738l5e7e31c2q4144bbd40e8c46b8@mail.gmail.com> <Pine.LNX.4.64.0710191733300.4086@zhemvz.fhfr.qr>

> > This makes sense to me.  If the type is still incomplete we don't want
> > to commit to consider its alias set as known and fixed.
> 
> Bootstrapped and tested on x86_64-unknown-linux-gnu for all languages
> including Ada.  The following is what I applied.

For the record, and since you wanted somebody familiar with the history of
this to comment, this seems fine.  We really shouldn't be doing anything
meaningful with the alias set of an incomplete type since nobody should be
allowed to reference such an object by the semantics of all languages.

References:
- [PATCH] Fix PRs 33816, 32921 (again, different)
  - From: Richard Guenther
- Re: [PATCH] Fix PRs 33816, 32921 (again, different)
  - From: Diego Novillo
- Re: [PATCH] Fix PRs 33816, 32921 (again, different)
  - From: Richard Guenther

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]