public inbox for gcc-patches@gcc.gnu.org
 help / color / mirror / Atom feed
* [PATCH, SMS 3/3] Skip DEBUG_INSN in loop-doloo​​​​p.
@ 2011-05-08  9:10 Revital1 Eres
  2011-05-08 12:02 ` Revital1 Eres
  0 siblings, 1 reply; 9+ messages in thread
From: Revital1 Eres @ 2011-05-08  9:10 UTC (permalink / raw)
  To: gcc-patches; +Cc: rakdver, Patch Tracking, zaks


Hello,

(sorry for multiple copies of this email)

This small fix was inserted to skip DEBUG_INSNs while
recognizing doloop pattern in loop-doloop.c file.  It's a fix
for the already approved do-loop patch (not in mainline yet,
http://gcc.gnu.org/ml/gcc-patches/2011-01/msg01718.html) in loop-doloop.c

The patch was tested together with the rest of the patches in this series
and on top of the patch to support do-loop for ARM (not yet in mainline,
but approved http://gcc.gnu.org/ml/gcc-patches/2011-01/msg01718.html).
On ppc64-redhat-linux regtest as well as bootstrap with SMS flags
enabling SMS also on loops with stage count 1.  Regtested on SPU.
On arm-linux-gnueabi regtseted on c,c++. Bootstrap c language with SMS
flags enabling SMS also on loops with stage count 1.

OK for mainline?

Thanks,
Revital

Changelog:

   * loop-doloop.c (doloop_condition_get): Use prev_nondebug_insn
      instead of PREV_INSN.



--- loop-doloop.c       2011-05-07 16:08:27.000000000 +0300
+++ loop-doloop_new.c   2011-05-07 16:07:48.000000000 +0300
@@ -151,7 +151,7 @@ doloop_condition_get (rtx doloop_pat)
         inc = XVECEXP (PATTERN (prev_insn), 0, 1);
       }
      else
-        inc = PATTERN (PREV_INSN (doloop_pat));
+        inc = PATTERN (prev_insn);
      /* We expect the condition to be of the form (reg != 0)  */
      cond = XEXP (SET_SRC (cmp), 0);
      if (GET_CODE (cond) != NE || XEXP (cond, 1) != const0_rtx)

^ permalink raw reply	[flat|nested] 9+ messages in thread
* [PATCH, SMS 3/3] Skip DEBUG_INSN in loop-doloo​​​​p.
  2011-05-08  9:10 [PATCH, SMS 3/3] Skip DEBUG_INSN in loop-doloo​​​​p Revital1 Eres
@ 2011-05-08 12:02 ` Revital1 Eres
  0 siblings, 0 replies; 9+ messages in thread
From: Revital1 Eres @ 2011-05-08 12:02 UTC (permalink / raw)
  To: gcc-patches; +Cc: rakdver, Patch Tracking, zaks


Hello,

(sorry for multiple copies of this email)

This small fix was inserted to skip DEBUG_INSNs while
recognizing doloop pattern in loop-doloop.c file.  It's a fix
for the already approved do-loop patch (not in mainline yet,
http://gcc.gnu.org/ml/gcc-patches/2011-01/msg01718.html) in loop-doloop.c

The patch was tested together with the rest of the patches in this series
and on top of the patch to support do-loop for ARM (not yet in mainline,
but approved http://gcc.gnu.org/ml/gcc-patches/2011-01/msg01718.html).
On ppc64-redhat-linux regtest as well as bootstrap with SMS flags
enabling SMS also on loops with stage count 1.  Regtested on SPU.
On arm-linux-gnueabi regtseted on c,c++. Bootstrap c language with SMS
flags enabling SMS also on loops with stage count 1.

OK for mainline?

Thanks,
Revital

Changelog:

   * loop-doloop.c (doloop_condition_get): Use prev_nondebug_insn
      instead of PREV_INSN.



--- loop-doloop.c       2011-05-07 16:08:27.000000000 +0300
+++ loop-doloop_new.c   2011-05-07 16:07:48.000000000 +0300
@@ -151,7 +151,7 @@ doloop_condition_get (rtx doloop_pat)
         inc = XVECEXP (PATTERN (prev_insn), 0, 1);
       }
      else
-        inc = PATTERN (PREV_INSN (doloop_pat));
+        inc = PATTERN (prev_insn);
      /* We expect the condition to be of the form (reg != 0)  */
      cond = XEXP (SET_SRC (cmp), 0);
      if (GET_CODE (cond) != NE || XEXP (cond, 1) != const0_rtx)

^ permalink raw reply	[flat|nested] 9+ messages in thread
* [PATCH, SMS 3/3] Skip DEBUG_INSN in loop-doloo​​​p.
  2011-05-08  7:41 [PATCH, SMS 3/3] Skip DEBUG_INSN in loop-doloo​​​p Revital Eres
  2011-05-08  8:48 ` Revital Eres
@ 2011-05-08 11:40 ` Revital Eres
  1 sibling, 0 replies; 9+ messages in thread
From: Revital Eres @ 2011-05-08 11:40 UTC (permalink / raw)
  To: gcc-patches; +Cc: rakdver, Patch Tracking, zaks

[-- Attachment #1: Type: text/plain, Size: 921 bytes --]

Hello,

(sorry for multiple copies of this email)

This small fix was inserted to skip DEBUG_INSNs while
recognizing doloop pattern in loop-doloop.c file.  It's a fix
for the already approved do-loop patch (not in mainline yet,
http://gcc.gnu.org/ml/gcc-patches/2011-01/msg01718.html) in loop-doloop.c

The patch was tested together with the rest of the patches in this series
and on top of the patch to support do-loop for ARM (not yet in mainline,
but approved http://gcc.gnu.org/ml/gcc-patches/2011-01/msg01718.html).
On ppc64-redhat-linux regtest as well as bootstrap with SMS flags
enabling SMS also on loops with stage count 1.  Regtested on SPU.
On arm-linux-gnueabi regtseted on c,c++. Bootstrap c language with SMS
flags enabling SMS also on loops with stage count 1.

OK for mainline?

Thanks,
Revital

Changelog:

    * loop-doloop.c (doloop_condition_get): Use prev_nondebug_insn
       instead of PREV_INSN.

[-- Attachment #2: patch_doloop_sms.txt --]
[-- Type: text/plain, Size: 513 bytes --]

--- loop-doloop.c       2011-05-07 16:08:27.000000000 +0300
+++ loop-doloop_new.c   2011-05-07 16:07:48.000000000 +0300
@@ -151,7 +151,7 @@ doloop_condition_get (rtx doloop_pat)
         inc = XVECEXP (PATTERN (prev_insn), 0, 1);
       }
      else
-        inc = PATTERN (PREV_INSN (doloop_pat));
+        inc = PATTERN (prev_insn);
      /* We expect the condition to be of the form (reg != 0)  */
      cond = XEXP (SET_SRC (cmp), 0);
      if (GET_CODE (cond) != NE || XEXP (cond, 1) != const0_rtx)

^ permalink raw reply	[flat|nested] 9+ messages in thread
* [PATCH, SMS 3/3] Skip DEBUG_INSN in loop-doloo​​p.
  2011-05-08  7:39 [PATCH, SMS 3/3] Skip DEBUG_INSN in loop-doloo​​p Revital Eres
  2011-05-08  7:49 ` Revital Eres
  2011-05-08  9:20 ` Revital Eres
@ 2011-05-08 10:57 ` Revital Eres
  2 siblings, 0 replies; 9+ messages in thread
From: Revital Eres @ 2011-05-08 10:57 UTC (permalink / raw)
  To: gcc-patches; +Cc: rakdver, Patch Tracking, zaks

Hello,

(sorry for multiple copies of this email)

This small fix was inserted to skip DEBUG_INSNs while
recognizing doloop pattern in loop-doloop.c file.  It's a fix
for the already approved do-loop patch (not in mainline yet,
http://gcc.gnu.org/ml/gcc-patches/2011-01/msg01718.html) in loop-doloop.c

The patch was tested together with the rest of the patches in this series
and on top of the patch to support do-loop for ARM (not yet in mainline,
but approved http://gcc.gnu.org/ml/gcc-patches/2011-01/msg01718.html).
On ppc64-redhat-linux regtest as well as bootstrap with SMS flags
enabling SMS also on loops with stage count 1.  Regtested on SPU.
On arm-linux-gnueabi regtseted on c,c++. Bootstrap c language with SMS
flags enabling SMS also on loops with stage count 1.

OK for mainline?

Thanks,
Revital

Changelog:

     * loop-doloop.c (doloop_condition_get): Use prev_nondebug_insn
        instead of PREV_INSN.

--- loop-doloop.c       2011-05-07 16:08:27.000000000 +0300
+++ loop-doloop_new.c   2011-05-07 16:07:48.000000000 +0300
@@ -151,7 +151,7 @@ doloop_condition_get (rtx doloop_pat)
          inc = XVECEXP (PATTERN (prev_insn), 0, 1);
        }
       else
-        inc = PATTERN (PREV_INSN (doloop_pat));
+        inc = PATTERN (prev_insn);
       /* We expect the condition to be of the form (reg != 0)  */
       cond = XEXP (SET_SRC (cmp), 0);
       if (GET_CODE (cond) != NE || XEXP (cond, 1) != const0_rtx)

^ permalink raw reply	[flat|nested] 9+ messages in thread
* [PATCH, SMS 3/3] Skip DEBUG_INSN in loop-doloo​​p.
  2011-05-08  7:39 [PATCH, SMS 3/3] Skip DEBUG_INSN in loop-doloo​​p Revital Eres
  2011-05-08  7:49 ` Revital Eres
@ 2011-05-08  9:20 ` Revital Eres
  2011-05-08 10:57 ` Revital Eres
  2 siblings, 0 replies; 9+ messages in thread
From: Revital Eres @ 2011-05-08  9:20 UTC (permalink / raw)
  To: gcc-patches; +Cc: rakdver, Patch Tracking, zaks

Hello,

(sorry for multiple copies of this email)

This small fix was inserted to skip DEBUG_INSNs while
recognizing doloop pattern in loop-doloop.c file.  It's a fix
for the already approved do-loop patch (not in mainline yet,
http://gcc.gnu.org/ml/gcc-patches/2011-01/msg01718.html) in loop-doloop.c

The patch was tested together with the rest of the patches in this series
and on top of the patch to support do-loop for ARM (not yet in mainline,
but approved http://gcc.gnu.org/ml/gcc-patches/2011-01/msg01718.html).
On ppc64-redhat-linux regtest as well as bootstrap with SMS flags
enabling SMS also on loops with stage count 1.  Regtested on SPU.
On arm-linux-gnueabi regtseted on c,c++. Bootstrap c language with SMS
flags enabling SMS also on loops with stage count 1.

OK for mainline?

Thanks,
Revital

Changelog:

     * loop-doloop.c (doloop_condition_get): Use prev_nondebug_insn
        instead of PREV_INSN.

--- loop-doloop.c       2011-05-07 16:08:27.000000000 +0300
+++ loop-doloop_new.c   2011-05-07 16:07:48.000000000 +0300
@@ -151,7 +151,7 @@ doloop_condition_get (rtx doloop_pat)
          inc = XVECEXP (PATTERN (prev_insn), 0, 1);
        }
       else
-        inc = PATTERN (PREV_INSN (doloop_pat));
+        inc = PATTERN (prev_insn);
       /* We expect the condition to be of the form (reg != 0)  */
       cond = XEXP (SET_SRC (cmp), 0);
       if (GET_CODE (cond) != NE || XEXP (cond, 1) != const0_rtx)

^ permalink raw reply	[flat|nested] 9+ messages in thread
* [PATCH, SMS 3/3] Skip DEBUG_INSN in loop-doloo​​​p.
  2011-05-08  7:41 [PATCH, SMS 3/3] Skip DEBUG_INSN in loop-doloo​​​p Revital Eres
@ 2011-05-08  8:48 ` Revital Eres
  2011-05-08 11:40 ` Revital Eres
  1 sibling, 0 replies; 9+ messages in thread
From: Revital Eres @ 2011-05-08  8:48 UTC (permalink / raw)
  To: gcc-patches; +Cc: rakdver, Patch Tracking, zaks

[-- Attachment #1: Type: text/plain, Size: 921 bytes --]

Hello,

(sorry for multiple copies of this email)

This small fix was inserted to skip DEBUG_INSNs while
recognizing doloop pattern in loop-doloop.c file.  It's a fix
for the already approved do-loop patch (not in mainline yet,
http://gcc.gnu.org/ml/gcc-patches/2011-01/msg01718.html) in loop-doloop.c

The patch was tested together with the rest of the patches in this series
and on top of the patch to support do-loop for ARM (not yet in mainline,
but approved http://gcc.gnu.org/ml/gcc-patches/2011-01/msg01718.html).
On ppc64-redhat-linux regtest as well as bootstrap with SMS flags
enabling SMS also on loops with stage count 1.  Regtested on SPU.
On arm-linux-gnueabi regtseted on c,c++. Bootstrap c language with SMS
flags enabling SMS also on loops with stage count 1.

OK for mainline?

Thanks,
Revital

Changelog:

    * loop-doloop.c (doloop_condition_get): Use prev_nondebug_insn
       instead of PREV_INSN.

[-- Attachment #2: patch_doloop_sms.txt --]
[-- Type: text/plain, Size: 513 bytes --]

--- loop-doloop.c       2011-05-07 16:08:27.000000000 +0300
+++ loop-doloop_new.c   2011-05-07 16:07:48.000000000 +0300
@@ -151,7 +151,7 @@ doloop_condition_get (rtx doloop_pat)
         inc = XVECEXP (PATTERN (prev_insn), 0, 1);
       }
      else
-        inc = PATTERN (PREV_INSN (doloop_pat));
+        inc = PATTERN (prev_insn);
      /* We expect the condition to be of the form (reg != 0)  */
      cond = XEXP (SET_SRC (cmp), 0);
      if (GET_CODE (cond) != NE || XEXP (cond, 1) != const0_rtx)

^ permalink raw reply	[flat|nested] 9+ messages in thread
* [PATCH, SMS 3/3] Skip DEBUG_INSN in loop-doloo​​p.
  2011-05-08  7:39 [PATCH, SMS 3/3] Skip DEBUG_INSN in loop-doloo​​p Revital Eres
@ 2011-05-08  7:49 ` Revital Eres
  2011-05-08  9:20 ` Revital Eres
  2011-05-08 10:57 ` Revital Eres
  2 siblings, 0 replies; 9+ messages in thread
From: Revital Eres @ 2011-05-08  7:49 UTC (permalink / raw)
  To: gcc-patches; +Cc: rakdver, Patch Tracking, zaks

Hello,

(sorry for multiple copies of this email)

This small fix was inserted to skip DEBUG_INSNs while
recognizing doloop pattern in loop-doloop.c file.  It's a fix
for the already approved do-loop patch (not in mainline yet,
http://gcc.gnu.org/ml/gcc-patches/2011-01/msg01718.html) in loop-doloop.c

The patch was tested together with the rest of the patches in this series
and on top of the patch to support do-loop for ARM (not yet in mainline,
but approved http://gcc.gnu.org/ml/gcc-patches/2011-01/msg01718.html).
On ppc64-redhat-linux regtest as well as bootstrap with SMS flags
enabling SMS also on loops with stage count 1.  Regtested on SPU.
On arm-linux-gnueabi regtseted on c,c++. Bootstrap c language with SMS
flags enabling SMS also on loops with stage count 1.

OK for mainline?

Thanks,
Revital

Changelog:

     * loop-doloop.c (doloop_condition_get): Use prev_nondebug_insn
        instead of PREV_INSN.

--- loop-doloop.c       2011-05-07 16:08:27.000000000 +0300
+++ loop-doloop_new.c   2011-05-07 16:07:48.000000000 +0300
@@ -151,7 +151,7 @@ doloop_condition_get (rtx doloop_pat)
          inc = XVECEXP (PATTERN (prev_insn), 0, 1);
        }
       else
-        inc = PATTERN (PREV_INSN (doloop_pat));
+        inc = PATTERN (prev_insn);
       /* We expect the condition to be of the form (reg != 0)  */
       cond = XEXP (SET_SRC (cmp), 0);
       if (GET_CODE (cond) != NE || XEXP (cond, 1) != const0_rtx)

^ permalink raw reply	[flat|nested] 9+ messages in thread
* [PATCH, SMS 3/3] Skip DEBUG_INSN in loop-doloo​​​p.
@ 2011-05-08  7:41 Revital Eres
  2011-05-08  8:48 ` Revital Eres
  2011-05-08 11:40 ` Revital Eres
  0 siblings, 2 replies; 9+ messages in thread
From: Revital Eres @ 2011-05-08  7:41 UTC (permalink / raw)
  To: gcc-patches; +Cc: rakdver, Patch Tracking, zaks

[-- Attachment #1: Type: text/plain, Size: 921 bytes --]

Hello,

(sorry for multiple copies of this email)

This small fix was inserted to skip DEBUG_INSNs while
recognizing doloop pattern in loop-doloop.c file.  It's a fix
for the already approved do-loop patch (not in mainline yet,
http://gcc.gnu.org/ml/gcc-patches/2011-01/msg01718.html) in loop-doloop.c

The patch was tested together with the rest of the patches in this series
and on top of the patch to support do-loop for ARM (not yet in mainline,
but approved http://gcc.gnu.org/ml/gcc-patches/2011-01/msg01718.html).
On ppc64-redhat-linux regtest as well as bootstrap with SMS flags
enabling SMS also on loops with stage count 1.  Regtested on SPU.
On arm-linux-gnueabi regtseted on c,c++. Bootstrap c language with SMS
flags enabling SMS also on loops with stage count 1.

OK for mainline?

Thanks,
Revital

Changelog:

    * loop-doloop.c (doloop_condition_get): Use prev_nondebug_insn
       instead of PREV_INSN.

[-- Attachment #2: patch_doloop_sms.txt --]
[-- Type: text/plain, Size: 513 bytes --]

--- loop-doloop.c       2011-05-07 16:08:27.000000000 +0300
+++ loop-doloop_new.c   2011-05-07 16:07:48.000000000 +0300
@@ -151,7 +151,7 @@ doloop_condition_get (rtx doloop_pat)
         inc = XVECEXP (PATTERN (prev_insn), 0, 1);
       }
      else
-        inc = PATTERN (PREV_INSN (doloop_pat));
+        inc = PATTERN (prev_insn);
      /* We expect the condition to be of the form (reg != 0)  */
      cond = XEXP (SET_SRC (cmp), 0);
      if (GET_CODE (cond) != NE || XEXP (cond, 1) != const0_rtx)

^ permalink raw reply	[flat|nested] 9+ messages in thread
* [PATCH, SMS 3/3] Skip DEBUG_INSN in loop-doloo​​p.
@ 2011-05-08  7:39 Revital Eres
  2011-05-08  7:49 ` Revital Eres
                   ` (2 more replies)
  0 siblings, 3 replies; 9+ messages in thread
From: Revital Eres @ 2011-05-08  7:39 UTC (permalink / raw)
  To: gcc-patches; +Cc: rakdver, Patch Tracking, zaks

Hello,

(sorry for multiple copies of this email)

This small fix was inserted to skip DEBUG_INSNs while
recognizing doloop pattern in loop-doloop.c file.  It's a fix
for the already approved do-loop patch (not in mainline yet,
http://gcc.gnu.org/ml/gcc-patches/2011-01/msg01718.html) in loop-doloop.c

The patch was tested together with the rest of the patches in this series
and on top of the patch to support do-loop for ARM (not yet in mainline,
but approved http://gcc.gnu.org/ml/gcc-patches/2011-01/msg01718.html).
On ppc64-redhat-linux regtest as well as bootstrap with SMS flags
enabling SMS also on loops with stage count 1.  Regtested on SPU.
On arm-linux-gnueabi regtseted on c,c++. Bootstrap c language with SMS
flags enabling SMS also on loops with stage count 1.

OK for mainline?

Thanks,
Revital

Changelog:

     * loop-doloop.c (doloop_condition_get): Use prev_nondebug_insn
        instead of PREV_INSN.

--- loop-doloop.c       2011-05-07 16:08:27.000000000 +0300
+++ loop-doloop_new.c   2011-05-07 16:07:48.000000000 +0300
@@ -151,7 +151,7 @@ doloop_condition_get (rtx doloop_pat)
          inc = XVECEXP (PATTERN (prev_insn), 0, 1);
        }
       else
-        inc = PATTERN (PREV_INSN (doloop_pat));
+        inc = PATTERN (prev_insn);
       /* We expect the condition to be of the form (reg != 0)  */
       cond = XEXP (SET_SRC (cmp), 0);
       if (GET_CODE (cond) != NE || XEXP (cond, 1) != const0_rtx)

^ permalink raw reply	[flat|nested] 9+ messages in thread
end of thread, other threads:[~2011-05-08  7:51 UTC | newest]

Thread overview: 9+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2011-05-08  9:10 [PATCH, SMS 3/3] Skip DEBUG_INSN in loop-doloo​​​​p Revital1 Eres
2011-05-08 12:02 ` Revital1 Eres
  -- strict thread matches above, loose matches on Subject: below --
2011-05-08  7:41 [PATCH, SMS 3/3] Skip DEBUG_INSN in loop-doloo​​​p Revital Eres
2011-05-08  8:48 ` Revital Eres
2011-05-08 11:40 ` Revital Eres
2011-05-08  7:39 [PATCH, SMS 3/3] Skip DEBUG_INSN in loop-doloo​​p Revital Eres
2011-05-08  7:49 ` Revital Eres
2011-05-08  9:20 ` Revital Eres
2011-05-08 10:57 ` Revital Eres

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).