Re: [PATCH] Fix up noce_try_abs again (PR rtl-optimization/68670)

2015-12-10 Thread Jakub Jelinek
On Thu, Dec 10, 2015 at 02:20:04PM +0100, Bernd Schmidt wrote: > This turns into a much simpler if statement, doesn't it? Ok with that > change. Ok, will do, thanks. > >+/* PR rtl-optimization/68670 */ > >+/* { dg-do run } */ > >+/* { dg-options "-O2 -ftracer" } */ > > Curious what effect -ftrac

Re: [PATCH] Fix up noce_try_abs again (PR rtl-optimization/68670)

2015-12-10 Thread Bernd Schmidt
On 12/09/2015 11:37 PM, Jakub Jelinek wrote: Not sure what I've been thinking when writing the previous noce_try_abs fix. I thought that the optimization can be applied for all the conditions, and whether it can be applied depends on if it is cond ? ~x : x or cond ? x : ~x. But that is not the c

[PATCH] Fix up noce_try_abs again (PR rtl-optimization/68670)

2015-12-09 Thread Jakub Jelinek
Hi! Not sure what I've been thinking when writing the previous noce_try_abs fix. I thought that the optimization can be applied for all the conditions, and whether it can be applied depends on if it is cond ? ~x : x or cond ? x : ~x. But that is not the case, the optimization can be only applied