Re: [PATCH PING 2/5] Named address spaces: support multiple pointer modes

["Ulrich Weigand" <uweigand@de.ibm.com>]

Richard Henderson wrote:
> On 09/21/2009 06:41 AM, Ulrich Weigand wrote:
> > --- gcc/alias.c	(working copy)
> > *************** along with GCC; see the file COPYING3.
> > *** 49,55 ****
> >    #include "tree-ssa-alias.h"
> >    #include "pointer-set.h"
> >    #include "tree-flow.h"
> > ! #include "targhooks.h"
> >
> >    /* The aliasing API provided here solves related but different problems:
> >
> > --- 49,55 ----
> >    #include "tree-ssa-alias.h"
> >    #include "pointer-set.h"
> >    #include "tree-flow.h"
> > ! #include "target-def.h"
> 
> 
> If you have to make this change, you've placed the macro
> in the wrong place.

Note that this patch applies to the branch; mainline currently
contains neither of the includes.  So this basically only changes
the resulting mainline patch into adding #include "target-def.h"
instead of adding #include "targhooks.h" ...

It seemed to me that of the two, target-def.h was a more natural
place for the new macro.  If you have a better suggestion where
to place it, I'll be happy to do so ...

Thanks,
Ulrich

-- 
  Dr. Ulrich Weigand
  GNU Toolchain for Linux on System z and Cell BE
  Ulrich.Weigand@de.ibm.com