Re: [PATCH] convert rs6000.c:branch_island_list to a VEC

[IainS <developer at sandoe-acoustics dot co dot uk>]

On 7 Aug 2010, at 19:40, David Edelsohn wrote:

> On Sat, Aug 7, 2010 at 1:53 PM, Nathan Froyd  
> <froydnj@codesourcery.com> wrote:
> > The patch below converts rs6000.c:branch_island_list to a VEC.  We'll
> > use slightly less space and warts like the int_cst bits go away as  
> > well.
> >
> > Unfortunately, I can't test the patch, as I don't have a PPC OS X
> > machine available.  Iain, could you please test the patch for me?

tested  -mmacosx-version-min=10.4 at m32 & m64 (save for ObjC* which  
are only tested at m32).
no java or ada.

manually checked {-m32,-m64}{,-mdynamic-no-pic} with -mmacosx-version- 
min=10.4 to see that the stubs change with the permutations.

AFAICT, no (new) problems ...

cheers,
Iain


> >  OK to
> > commit if testing looks good?
> >
> > -Nathan
> >
> >        * config/rs6000/rs6000.c (branch_island): Define.  Define a  
> > VEC
> >        of it.
> >        (branch_island_list): Delete.
> >        (branch_islands): Declare.
> >        (add_compiler_branch_island): Adjust for branch_islands  
> > instead of
> >        branch_island_list.
> >        (macho_branch_islands): Likewise.
> >        (no_previous_def): Likewise.
> >        (get_prev_label): Likewise.
>
> Okay.
>
> Thanks, David