Re: [PATCH 1/2] Allow REG_EQUAL for ZERO_EXTRACT

2015-08-03 Thread Jeff Law
On 07/28/2015 06:00 AM, Kugan wrote: On 27/07/15 05:38, Andreas Schwab wrote: Kugan writes: * cse.c (cse_insn): Fix missing check for STRICT_LOW_PART and minor clean up. This breaks gcc.target/m68k/tls-ie-xgot.c scan-assembler jsr __m68k_read_tp gcc.target/m68k/tls-ie.c s

Re: [PATCH 1/2] Allow REG_EQUAL for ZERO_EXTRACT

2015-07-30 Thread Andreas Schwab
Kugan writes: > * cse.c (cse_insn): Restoring old behaviour for src_eqv >when dest and value in the REG_EQUAL are same and dest >is STRICT_LOW_PART. This fixes the regression and doesn't introduce any new one. Andreas. -- Andreas Schwab, sch...@linux-m68k.org GPG Key fin

Re: [PATCH 1/2] Allow REG_EQUAL for ZERO_EXTRACT

2015-07-28 Thread Kugan
On 27/07/15 05:38, Andreas Schwab wrote: > Kugan writes: > >> * cse.c (cse_insn): Fix missing check for STRICT_LOW_PART and minor >> clean up. > > This breaks > > gcc.target/m68k/tls-ie-xgot.c scan-assembler jsr __m68k_read_tp > gcc.target/m68k/tls-ie.c scan-assembler jsr __m68k_re

Re: [PATCH 1/2] Allow REG_EQUAL for ZERO_EXTRACT

2015-07-26 Thread Kugan
On 27/07/15 05:38, Andreas Schwab wrote: > Kugan writes: > >> * cse.c (cse_insn): Fix missing check for STRICT_LOW_PART and minor >> clean up. > > This breaks > > gcc.target/m68k/tls-ie-xgot.c scan-assembler jsr __m68k_read_tp > gcc.target/m68k/tls-ie.c scan-assembler jsr __m68k_re

Re: [PATCH 1/2] Allow REG_EQUAL for ZERO_EXTRACT

2015-07-26 Thread Andreas Schwab
Kugan writes: > * cse.c (cse_insn): Fix missing check for STRICT_LOW_PART and minor > clean up. This breaks gcc.target/m68k/tls-ie-xgot.c scan-assembler jsr __m68k_read_tp gcc.target/m68k/tls-ie.c scan-assembler jsr __m68k_read_tp gcc.target/m68k/tls-le-xtls.c scan-assembler jsr __

Re: [PATCH 1/2] Allow REG_EQUAL for ZERO_EXTRACT

2015-07-23 Thread Jeff Law
On 07/19/2015 09:17 PM, Kugan wrote: I have made a mistake while addressing the review comments for this patch. Unfortunately, It was not detected in my earlier testing. My sincere graphology for the mistake. I have basically missed the STRICT_LOW_PART check for the first if-check thus the secon

Re: [PATCH 1/2] Allow REG_EQUAL for ZERO_EXTRACT

2015-07-19 Thread Kugan
I have made a mistake while addressing the review comments for this patch. Unfortunately, It was not detected in my earlier testing. My sincere graphology for the mistake. I have basically missed the STRICT_LOW_PART check for the first if-check thus the second part (which is the ZERO_EXTRACT part)

Re: [PATCH 1/2] Allow REG_EQUAL for ZERO_EXTRACT

2015-07-06 Thread Jeff Law
On 07/05/2015 05:16 PM, Kugan wrote: diff --git a/gcc/ChangeLog b/gcc/ChangeLog index fc23abd..5796f16 100644 --- a/gcc/ChangeLog +++ b/gcc/ChangeLog @@ -1,3 +1,10 @@ + +2015-06-26 Kugan Vivekanandarajah + + * cse.c (cse_insn): Calculate src_eqv for ZERO_EXTRACT. + * emit-rtl.c (set

Re: [PATCH 1/2] Allow REG_EQUAL for ZERO_EXTRACT

2015-07-05 Thread Kugan
On 30/06/15 17:16, Maxim Kuvyrkov wrote: >> On Jun 30, 2015, at 6:54 AM, Kugan wrote: >> >> >> On 29/06/15 21:56, Maxim Kuvyrkov wrote: On Jun 28, 2015, at 2:28 PM, Kugan wrote: This patch allows setting REG_EQUAL for ZERO_EXTRACT and handle that in cse (where the src

Re: [PATCH 1/2] Allow REG_EQUAL for ZERO_EXTRACT

2015-06-30 Thread Maxim Kuvyrkov
> On Jun 30, 2015, at 6:54 AM, Kugan wrote: > > > On 29/06/15 21:56, Maxim Kuvyrkov wrote: >>> On Jun 28, 2015, at 2:28 PM, Kugan >>> wrote: >>> >>> This patch allows setting REG_EQUAL for ZERO_EXTRACT and handle that in >>> cse (where the src for the ZERO_EXTRACT needs to be calculated) >>>

Re: [PATCH 1/2] Allow REG_EQUAL for ZERO_EXTRACT

2015-06-29 Thread Kugan
On 29/06/15 21:56, Maxim Kuvyrkov wrote: >> On Jun 28, 2015, at 2:28 PM, Kugan wrote: >> >> This patch allows setting REG_EQUAL for ZERO_EXTRACT and handle that in >> cse (where the src for the ZERO_EXTRACT needs to be calculated) >> >> Thanks, >> Kugan > >> From 75e746e559ffd21b25542b3db627e3b3

Re: [PATCH 1/2] Allow REG_EQUAL for ZERO_EXTRACT

2015-06-29 Thread Maxim Kuvyrkov
> On Jun 28, 2015, at 2:28 PM, Kugan wrote: > > This patch allows setting REG_EQUAL for ZERO_EXTRACT and handle that in > cse (where the src for the ZERO_EXTRACT needs to be calculated) > > Thanks, > Kugan > From 75e746e559ffd21b25542b3db627e3b318118569 Mon Sep 17 00:00:00 2001 > From: Kugan Vi

[PATCH 1/2] Allow REG_EQUAL for ZERO_EXTRACT

2015-06-28 Thread Kugan
This patch allows setting REG_EQUAL for ZERO_EXTRACT and handle that in cse (where the src for the ZERO_EXTRACT needs to be calculated) Thanks, Kugan 2015-06-26 Kugan Vivekanandarajah * cse.c (cse_insn): Calculate src_eqv for ZERO_EXTRACT. * emit-rtl.c (set_for_reg_notes): Al