Re: ipa: Fix removal of multi-target speculation

2020-01-29 Thread Martin Liška
On 1/29/20 1:39 PM, Jan Hubicka wrote: diff --git a/gcc/testsuite/gcc.dg/tree-prof/indir-call-prof-2.c b/gcc/testsuite/gcc.dg/tree-prof/indir-call-prof-2.c index 61612b5b628..bbba0521018 100644 --- a/gcc/testsuite/gcc.dg/tree-prof/indir-call-prof-2.c +++ b/gcc/testsuite/gcc.dg/tree-prof/indir-ca

Re: ipa: Fix removal of multi-target speculation

2020-01-29 Thread Jan Hubicka
> > diff --git a/gcc/testsuite/gcc.dg/tree-prof/indir-call-prof-2.c > > b/gcc/testsuite/gcc.dg/tree-prof/indir-call-prof-2.c > > index 61612b5b628..bbba0521018 100644 > > --- a/gcc/testsuite/gcc.dg/tree-prof/indir-call-prof-2.c > > +++ b/gcc/testsuite/gcc.dg/tree-prof/indir-call-prof-2.c > > This

Re: ipa: Fix removal of multi-target speculation

2020-01-29 Thread Martin Liška
On 1/29/20 12:46 PM, Jan Hubicka wrote: Hi, this fixes indirect call speculation ICE Martin (Liska) reduced from spec GCC. The problem here was in resolve_speculation that if called to resolve the second edge in multi-target speculative call resolved first on instead. This is bug I did not catc

ipa: Fix removal of multi-target speculation

2020-01-29 Thread Jan Hubicka
Hi, this fixes indirect call speculation ICE Martin (Liska) reduced from spec GCC. The problem here was in resolve_speculation that if called to resolve the second edge in multi-target speculative call resolved first on instead. This is bug I did not catch during the API change. Other problem is