Re: [PATCH] Do not simplify "(and (reg) (const bit))" to if_then_else.

2016-12-01 Thread Jeff Law
On 11/21/2016 05:36 AM, Dominik Vogt wrote: On Fri, Nov 11, 2016 at 12:10:28PM +0100, Dominik Vogt wrote: > On Mon, Nov 07, 2016 at 09:29:26PM +0100, Bernd Schmidt wrote: > > On 10/31/2016 08:56 PM, Dominik Vogt wrote: > > > > >combine_simplify_rtx() tries to replace rtx expressions with just

Re: [PATCH] Do not simplify "(and (reg) (const bit))" to if_then_else.

2016-12-01 Thread Bernd Schmidt
On 11/21/2016 01:36 PM, Dominik Vogt wrote: diff --git a/gcc/combine.c b/gcc/combine.c index b22a274..457fe8a 100644 --- a/gcc/combine.c +++ b/gcc/combine.c @@ -5575,10 +5575,23 @@ combine_simplify_rtx (rtx x, machine_mode op0_mode, int in_dest, { rtx cop1 = const0_rtx;

Re: [PATCH] Do not simplify "(and (reg) (const bit))" to if_then_else.

2016-11-21 Thread Dominik Vogt
On Fri, Nov 11, 2016 at 12:10:28PM +0100, Dominik Vogt wrote: > On Mon, Nov 07, 2016 at 09:29:26PM +0100, Bernd Schmidt wrote: > > On 10/31/2016 08:56 PM, Dominik Vogt wrote: > > > > >combine_simplify_rtx() tries to replace rtx expressions with just two > > >possible values with an experession

Re: [PATCH] Do not simplify "(and (reg) (const bit))" to if_then_else.

2016-11-11 Thread Dominik Vogt
On Mon, Nov 07, 2016 at 09:29:26PM +0100, Bernd Schmidt wrote: > On 10/31/2016 08:56 PM, Dominik Vogt wrote: > > >combine_simplify_rtx() tries to replace rtx expressions with just two > >possible values with an experession that uses if_then_else: > > > > (if_then_else (condition) (value1)

Re: [PATCH] Do not simplify "(and (reg) (const bit))" to if_then_else.

2016-11-07 Thread Bernd Schmidt
On 10/31/2016 08:56 PM, Dominik Vogt wrote: combine_simplify_rtx() tries to replace rtx expressions with just two possible values with an experession that uses if_then_else: (if_then_else (condition) (value1) (value2)) If the original expression is e.g. (and (reg) (const_int 2)) I'm

[PING, PATCH] Do not simplify "(and (reg) (const bit))" to if_then_else.

2016-11-07 Thread Dominik Vogt
Ping. https://gcc.gnu.org/ml/gcc-patches/2016-10/msg02525.html On Mon, Oct 31, 2016 at 08:56:10PM +0100, Dominik Vogt wrote: > The attached patch does a little change in > combine.c:combine_simplify_rtx() to prevent a "simplification" > where the rtl code gets more complex in reality. The

[PATCH] Do not simplify "(and (reg) (const bit))" to if_then_else.

2016-10-31 Thread Dominik Vogt
The attached patch does a little change in combine.c:combine_simplify_rtx() to prevent a "simplification" where the rtl code gets more complex in reality. The complete description of the change can be found in the commit comment in the attached patch. The patch reduces the number of patterns in