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: [tree-ssa, mainline] Fix nested inlining

> On 18 Oct 2003 16:21:06 -0300, Alexandre Oliva <aoliva@redhat.com> wrote:
> 
> > On Oct 18, 2003, Jan Hubicka <hubicka@ucw.cz> wrote:
> >
> >> -   DECL_ABSTRACT_ORIGIN (copy) = DECL_ORIGIN (decl);
> >   
> >> +   if (DECL_ABSTRACT_ORIGIN (decl))
> >> +     DECL_ABSTRACT_ORIGIN (copy) = DECL_ABSTRACT_ORIGIN (decl);
> >> +   else
> >> +     DECL_ABSTRACT_ORIGIN (copy) = DECL_ORIGIN (decl);
> >   
> > Err...  Given the definition of DECL_ORIGIN, I don't see how you
> > change could make sense.  For reference:
> 
> Agreed, this patch is not OK.
I already reverted this.
Sorry for that - I had several changes in my tree in order to get -O3
-funit-at-a-time bootstrap pass and didn't notices that this is NOOP and
not needed.

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