Re: [patch 10/10] debug-early merge: compiler proper

[Aldy Hernandez <aldyh at redhat dot com>]

On 05/22/2015 07:26 AM, Richard Biener wrote:
> On Wed, May 20, 2015 at 11:45 PM, Aldy Hernandez <aldyh@redhat.com> wrote:
> > On 05/20/2015 05:01 PM, Jan Hubicka wrote:
> > > > commit 8824b5ecba26cef065e47b34609c72677c3c36fc
> > > > Author: Aldy Hernandez <aldyh@redhat.com>
> > > > Date:   Wed May 20 16:31:14 2015 -0400
> > > >
> > > >       Set DECL_IGNORED_P on temporary arrays created in the switch
> > > >       conversion pass.
> > > >
> > > > diff --git a/gcc/tree-switch-conversion.c b/gcc/tree-switch-conversion.c
> > > > index 6b68a16..a4bcdba 100644
> > > > --- a/gcc/tree-switch-conversion.c
> > > > +++ b/gcc/tree-switch-conversion.c
> > > > @@ -1097,6 +1097,7 @@ build_one_array (gswitch *swtch, int num, tree
> > > > arr_index_type,
> > > >          DECL_ARTIFICIAL (decl) = 1;
> > > >          TREE_CONSTANT (decl) = 1;
> > > >          TREE_READONLY (decl) = 1;
> > > > +      DECL_IGNORED_P (decl) = 1;
> > > >          varpool_node::finalize_decl (decl);
> > >
> > >
> > > This looks obvious enough to me.  Technically speaking the array type
> > > constructed
> > > probalby should be TREE_ARTIFICAIL, but probably it does not matter.
>
> Fine to commit to trunk btw.

Tested independently on trunk, and committed there.

Thanks.