> The bug in the walker loop here IMO predated C++ conversion, but to
> avoid member function freeing THIS we probably want to have:
>
> cgraph_edge::remove (e),
> symtab_node::remove (e),
> ipa_ref::remove (ref),
> cgraph_node::resolve_speculation (e, target)
Actually thinking of this, perhas rem
> Hi,
>
> On Fri, Nov 15 2019, luoxhu wrote:
> >
> > Sorry to break the bootstrap. This was my lack of consideration when
> > splitting this small piece of code from the previous patch, the line
> > should be in the first line of the second loop. Could you please add
> > some comments that edge
Hi,
On Fri, Nov 15 2019, luoxhu wrote:
>
> Sorry to break the bootstrap. This was my lack of consideration when
> splitting this small piece of code from the previous patch, the line
> should be in the first line of the second loop. Could you please add
> some comments that edge may be freed in
On 2019/11/15 17:19, Jan Hubicka wrote:
>> On Fri, Nov 15, 2019 at 9:10 AM Jan Hubicka wrote:
>>>
next is initialized only in the loop before, it is never updated
in it's own loop.
gcc/ChangeLog
2019-11-15 Xiong Hu Luo
* ipa-inline.c (inl
> On Fri, Nov 15, 2019 at 10:16 AM Richard Biener
> wrote:
> >
> > On Fri, Nov 15, 2019 at 9:10 AM Jan Hubicka wrote:
> > >
> > > > next is initialized only in the loop before, it is never updated
> > > > in it's own loop.
> > > >
> > > > gcc/ChangeLog
> > > >
> > > > 2019-11-15 Xiong Hu L
> On Fri, Nov 15, 2019 at 9:10 AM Jan Hubicka wrote:
> >
> > > next is initialized only in the loop before, it is never updated
> > > in it's own loop.
> > >
> > > gcc/ChangeLog
> > >
> > > 2019-11-15 Xiong Hu Luo
> > >
> > > * ipa-inline.c (inline_small_functions): Update iterator
On Fri, Nov 15, 2019 at 10:16 AM Richard Biener
wrote:
>
> On Fri, Nov 15, 2019 at 9:10 AM Jan Hubicka wrote:
> >
> > > next is initialized only in the loop before, it is never updated
> > > in it's own loop.
> > >
> > > gcc/ChangeLog
> > >
> > > 2019-11-15 Xiong Hu Luo
> > >
> > >
On Fri, Nov 15, 2019 at 9:10 AM Jan Hubicka wrote:
>
> > next is initialized only in the loop before, it is never updated
> > in it's own loop.
> >
> > gcc/ChangeLog
> >
> > 2019-11-15 Xiong Hu Luo
> >
> > * ipa-inline.c (inline_small_functions): Update iterator of next.
>
> OK,
> t
> next is initialized only in the loop before, it is never updated
> in it's own loop.
>
> gcc/ChangeLog
>
> 2019-11-15 Xiong Hu Luo
>
> * ipa-inline.c (inline_small_functions): Update iterator of next.
OK,
thanks!
Honza
> ---
> gcc/ipa-inline.c | 15 +--
> 1 file ch
