public inbox for gcc-bugs@sourceware.org
help / color / mirror / Atom feed
From: "vries at gcc dot gnu.org" <gcc-bugzilla@gcc.gnu.org>
To: gcc-bugs@gcc.gnu.org
Subject: [Bug tree-optimization/66652] try_transform_to_exit_first_loop_alt generates incorrect loop
Date: Tue, 30 Jun 2015 09:58:00 -0000	[thread overview]
Message-ID: <bug-66652-4-N393NQjZst@http.gcc.gnu.org/bugzilla/> (raw)
In-Reply-To: <bug-66652-4@http.gcc.gnu.org/bugzilla/>

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66652

--- Comment #7 from vries at gcc dot gnu.org ---
Author: vries
Date: Tue Jun 30 09:57:20 2015
New Revision: 225166

URL: https://gcc.gnu.org/viewcvs?rev=225166&root=gcc&view=rev
Log:
Backport transform_to_exit_first_loop_alt-related patches

2015-06-30  Tom de Vries  <tom@codesourcery.com>

        backport from trunk:
        2015-06-30  Tom de Vries  <tom@codesourcery.com>

        PR tree-optimization/66652
        * tree-parloops.c (try_transform_to_exit_first_loop_alt): Use
        max_loop_iterations to determine if nit + 1 overflows.

        * testsuite/libgomp.c/parloops-exit-first-loop-alt-3.c (f): Rewrite
        using restrict pointers.
        (main): Add arguments to calls to f.
        * testsuite/libgomp.c/parloops-exit-first-loop-alt.c: Same.

        * gcc.dg/parloops-exit-first-loop-alt-pr66652.c: New test.
        * gcc.dg/parloops-exit-first-loop-alt-3.c (f):  Rewrite using restrict
        pointers.
        * gcc.dg/parloops-exit-first-loop-alt.c: Same.

        2015-06-29  Tom de Vries  <tom@codesourcery.com>

        * tree-parloops.c (try_transform_to_exit_first_loop_alt): Simplify
        function structure.

        2015-06-23  Tom de Vries  <tom@codesourcery.com>

        * testsuite/libgomp.c/parloops-exit-first-loop-alt-3.c (main): Use
        abort.
        * testsuite/libgomp.c/parloops-exit-first-loop-alt-4.c (main): Same.

        2015-06-22  Tom de Vries  <tom@codesourcery.com>

        * tree-parloops.c (transform_to_exit_first_loop_alt): Add update_stmt
        for cond_stmt.

        2015-06-13  Tom de Vries  <tom@codesourcery.com>

        * gcc.dg/parloops-exit-first-loop-alt-4.c: New test.

        * testsuite/libgomp.c/parloops-exit-first-loop-alt-4.c: New test.

        * testsuite/libgomp.c/parloops-exit-first-loop-alt-2.c: Add comment.
        * testsuite/libgomp.c/parloops-exit-first-loop-alt.c: Same.
        * testsuite/libgomp.c/parloops-exit-first-loop-alt-3.c: Add comment.
        (N): Define.
        (main): Use N instead of hardcoded constants.

        * gcc.dg/parloops-exit-first-loop-alt-2.c: Add comment.
        (main): Remove superfluous attributes.
        * gcc.dg/parloops-exit-first-loop-alt-3.c: Same.
        * gcc.dg/parloops-exit-first-loop-alt.c: Same.

Added:
   
branches/gomp-4_0-branch/gcc/testsuite/gcc.dg/parloops-exit-first-loop-alt-4.c
      - copied, changed from r225165,
branches/gomp-4_0-branch/gcc/testsuite/gcc.dg/parloops-exit-first-loop-alt-3.c
   
branches/gomp-4_0-branch/gcc/testsuite/gcc.dg/parloops-exit-first-loop-alt-pr66652.c
   
branches/gomp-4_0-branch/libgomp/testsuite/libgomp.c/parloops-exit-first-loop-alt-4.c
      - copied, changed from r225165,
branches/gomp-4_0-branch/libgomp/testsuite/libgomp.c/parloops-exit-first-loop-alt-3.c
Modified:
    branches/gomp-4_0-branch/gcc/ChangeLog.gomp
    branches/gomp-4_0-branch/gcc/testsuite/ChangeLog.gomp
   
branches/gomp-4_0-branch/gcc/testsuite/gcc.dg/parloops-exit-first-loop-alt-2.c
   
branches/gomp-4_0-branch/gcc/testsuite/gcc.dg/parloops-exit-first-loop-alt-3.c
   
branches/gomp-4_0-branch/gcc/testsuite/gcc.dg/parloops-exit-first-loop-alt.c
    branches/gomp-4_0-branch/gcc/tree-parloops.c
    branches/gomp-4_0-branch/libgomp/ChangeLog.gomp
   
branches/gomp-4_0-branch/libgomp/testsuite/libgomp.c/parloops-exit-first-loop-alt-2.c
   
branches/gomp-4_0-branch/libgomp/testsuite/libgomp.c/parloops-exit-first-loop-alt-3.c
   
branches/gomp-4_0-branch/libgomp/testsuite/libgomp.c/parloops-exit-first-loop-alt.c


      parent reply	other threads:[~2015-06-30  9:58 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <bug-66652-4@http.gcc.gnu.org/bugzilla/>
2015-06-25 12:37 ` vries at gcc dot gnu.org
2015-06-27  5:52 ` vries at gcc dot gnu.org
2015-06-29 11:41 ` vries at gcc dot gnu.org
2015-06-29 14:41 ` vries at gcc dot gnu.org
2015-06-30  8:36 ` vries at gcc dot gnu.org
2015-06-30  8:39 ` vries at gcc dot gnu.org
2015-06-30  9:58 ` vries at gcc dot gnu.org [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=bug-66652-4-N393NQjZst@http.gcc.gnu.org/bugzilla/ \
    --to=gcc-bugzilla@gcc.gnu.org \
    --cc=gcc-bugs@gcc.gnu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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).