Re: [PATCH] Fix PR78515

2017-01-21 Thread Richard Biener
On January 20, 2017 11:27:04 PM GMT+01:00, Martin Jambor wrote: >Hi, > >On Fri, Nov 25, 2016 at 02:55:29PM +0100, Richard Biener wrote: >> On Fri, 25 Nov 2016, Martin Jambor wrote: >> >> ... >> >> > > There's still that odd 'stmt2' >> > > hanging around that gets set to sth else than stmt with >>

Re: [PATCH] Fix PR78515

2017-01-20 Thread Martin Jambor
Hi, On Fri, Nov 25, 2016 at 02:55:29PM +0100, Richard Biener wrote: > On Fri, 25 Nov 2016, Martin Jambor wrote: > > ... > > > > There's still that odd 'stmt2' > > > hanging around that gets set to sth else than stmt with > > > > > > op1 = gimple_assign_rhs1 (stmt); > > > > > > if (TREE_CODE

Re: [PATCH] Fix PR78515

2016-12-15 Thread David Edelsohn
On Thu, Dec 15, 2016 at 3:23 AM, Jakub Jelinek wrote: > On Wed, Dec 14, 2016 at 05:10:23PM -0700, Martin Sebor wrote: >> The regression test is failing on powerpc64le due to the warnings >> below: >> >> FAIL: gcc.dg/torture/pr78515.c -O0 (test for excess errors) >> Excess errors: >> /src/gcc/tr

Re: [PATCH] Fix PR78515

2016-12-15 Thread Jakub Jelinek
On Wed, Dec 14, 2016 at 05:10:23PM -0700, Martin Sebor wrote: > The regression test is failing on powerpc64le due to the warnings > below: > > FAIL: gcc.dg/torture/pr78515.c -O0 (test for excess errors) > Excess errors: > /src/gcc/trunk/gcc/testsuite/gcc.dg/torture/pr78515.c:11:1: warning: GCC

Re: [PATCH] Fix PR78515

2016-12-14 Thread Martin Sebor
On 11/25/2016 06:55 AM, Richard Biener wrote: On Fri, 25 Nov 2016, Martin Jambor wrote: Hi, On Fri, Nov 25, 2016 at 12:01:38PM +0100, Richard Biener wrote: I am testing the following to beat some sanity into compute_complex_assign_jump_func. That the function does not handle ternary operat

Re: [PATCH] Fix PR78515

2016-11-25 Thread Richard Biener
On Fri, 25 Nov 2016, Martin Jambor wrote: > Hi, > > On Fri, Nov 25, 2016 at 12:01:38PM +0100, Richard Biener wrote: > > > > I am testing the following to beat some sanity into > > compute_complex_assign_jump_func. > > That the function does not handle ternary operations (did we have them > sin

Re: [PATCH] Fix PR78515

2016-11-25 Thread Martin Jambor
Hi, On Fri, Nov 25, 2016 at 12:01:38PM +0100, Richard Biener wrote: > > I am testing the following to beat some sanity into > compute_complex_assign_jump_func. That the function does not handle ternary operations (did we have them since the beginning?) is clearly my fault and the patch is fine.

[PATCH] Fix PR78515

2016-11-25 Thread Richard Biener
I am testing the following to beat some sanity into compute_complex_assign_jump_func. There's still that odd 'stmt2' hanging around that gets set to sth else than stmt with op1 = gimple_assign_rhs1 (stmt); if (TREE_CODE (op1) == SSA_NAME) { if (SSA_NAME_IS_DEFAULT_DEF (op1))