Re: Ping : [PATCH] [gcc, combine] PR46164: Don't combine the insns if a volatile register is contained.

2015-02-26 Thread Terry Guo
On Thu, Feb 26, 2015 at 1:55 PM, Segher Boessenkool seg...@kernel.crashing.org wrote: On Tue, Feb 17, 2015 at 11:39:34AM +0800, Terry Guo wrote: On Sun, Feb 15, 2015 at 7:35 PM, Segher Boessenkool seg...@kernel.crashing.org wrote: Hi Terry, I still think this is stage1 material. + /*

Re: Ping : [PATCH] [gcc, combine] PR46164: Don't combine the insns if a volatile register is contained.

2015-02-25 Thread Segher Boessenkool
On Tue, Feb 17, 2015 at 11:39:34AM +0800, Terry Guo wrote: On Sun, Feb 15, 2015 at 7:35 PM, Segher Boessenkool seg...@kernel.crashing.org wrote: Hi Terry, I still think this is stage1 material. + /* Don't combine if dest contains a user specified register and i3 contains +

Re: Ping : [PATCH] [gcc, combine] PR46164: Don't combine the insns if a volatile register is contained.

2015-02-16 Thread Terry Guo
On Sun, Feb 15, 2015 at 7:35 PM, Segher Boessenkool seg...@kernel.crashing.org wrote: Hi Terry, I still think this is stage1 material. + /* Don't combine if dest contains a user specified register and i3 contains + ASM_OPERANDS, because the user specified register (same with dest) in

Re: Ping : [PATCH] [gcc, combine] PR46164: Don't combine the insns if a volatile register is contained.

2015-02-15 Thread Terry Guo
On Fri, Feb 13, 2015 at 5:06 PM, Richard Sandiford rdsandif...@googlemail.com wrote: Segher Boessenkool seg...@kernel.crashing.org writes: On Thu, Feb 12, 2015 at 03:54:21PM +, Richard Sandiford wrote: Hale Wang hale.w...@arm.com writes: Ping? It's not a regression (or is it?), so it is

Re: Ping : [PATCH] [gcc, combine] PR46164: Don't combine the insns if a volatile register is contained.

2015-02-15 Thread Segher Boessenkool
Hi Terry, I still think this is stage1 material. + /* Don't combine if dest contains a user specified register and i3 contains + ASM_OPERANDS, because the user specified register (same with dest) in i3 + would be replaced by the src of insn which might be different with + the

Re: Ping : [PATCH] [gcc, combine] PR46164: Don't combine the insns if a volatile register is contained.

2015-02-13 Thread Richard Sandiford
Segher Boessenkool seg...@kernel.crashing.org writes: On Thu, Feb 12, 2015 at 03:54:21PM +, Richard Sandiford wrote: Hale Wang hale.w...@arm.com writes: Ping? It's not a regression (or is it?), so it is not appropriate for stage4. diff --git a/gcc/combine.c b/gcc/combine.c index

Re: Ping : [PATCH] [gcc, combine] PR46164: Don't combine the insns if a volatile register is contained.

2015-02-12 Thread Segher Boessenkool
On Thu, Feb 12, 2015 at 03:54:21PM +, Richard Sandiford wrote: Hale Wang hale.w...@arm.com writes: Ping? It's not a regression (or is it?), so it is not appropriate for stage4. diff --git a/gcc/combine.c b/gcc/combine.c index 5c763b4..6901ac2 100644 --- a/gcc/combine.c +++

Re: Ping : [PATCH] [gcc, combine] PR46164: Don't combine the insns if a volatile register is contained.

2015-02-12 Thread Jeff Law
On 02/12/15 15:15, Segher Boessenkool wrote: On Thu, Feb 12, 2015 at 03:54:21PM +, Richard Sandiford wrote: Hale Wang hale.w...@arm.com writes: Ping? It's not a regression (or is it?), so it is not appropriate for stage4. That's the big question, of course. [ INSN can never be an asm,

RE: Ping : [PATCH] [gcc, combine] PR46164: Don't combine the insns if a volatile register is contained.

2015-02-12 Thread Hale Wang
-Original Message- From: Segher Boessenkool [mailto:seg...@kernel.crashing.org] Sent: Friday, February 13, 2015 6:16 AM To: Hale Wang; 'GCC Patches'; Richard Sandiford Subject: Re: Ping : [PATCH] [gcc, combine] PR46164: Don't combine the insns if a volatile register is contained.

Re: Ping : [PATCH] [gcc, combine] PR46164: Don't combine the insns if a volatile register is contained.

2015-02-12 Thread Richard Sandiford
Hale Wang hale.w...@arm.com writes: Ping? -Original Message- From: Hale Wang [mailto:hale.w...@arm.com] Sent: Thursday, January 29, 2015 9:58 AM To: Hale Wang; 'Segher Boessenkool' Cc: GCC Patches Subject: RE: [PATCH] [gcc, combine] PR46164: Don't combine the insns if a volatile

Ping : [PATCH] [gcc, combine] PR46164: Don't combine the insns if a volatile register is contained.

2015-02-08 Thread Hale Wang
Ping? -Original Message- From: Hale Wang [mailto:hale.w...@arm.com] Sent: Thursday, January 29, 2015 9:58 AM To: Hale Wang; 'Segher Boessenkool' Cc: GCC Patches Subject: RE: [PATCH] [gcc, combine] PR46164: Don't combine the insns if a volatile register is contained. Hi Segher,