public inbox for fortran@gcc.gnu.org
 help / color / mirror / Atom feed
From: Tobias Burnus <tobias@codesourcery.com>
To: Jakub Jelinek <jakub@redhat.com>
Cc: gcc-patches <gcc-patches@gcc.gnu.org>, fortran <fortran@gcc.gnu.org>
Subject: Re: [Patch] OpenMP: Add iterator support to Fortran's depend; add affinity clause
Date: Mon, 31 May 2021 16:43:21 +0200	[thread overview]
Message-ID: <3995781f-b673-df3c-12f8-a0635867ae43@codesourcery.com> (raw)
In-Reply-To: <20210529080310.GZ7746@tucnak>

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

On 29.05.21 10:03, Jakub Jelinek wrote:
> On Fri, May 28, 2021 at 12:59:20AM +0200, Tobias Burnus wrote:
>>      * gfortran.dg/gomp/depend-iterator-1.f90: New test.
>>      * gfortran.dg/gomp/depend-iterator-2.f90: New test.
> Something I've missed during the review but it shows up during testing:

Now fixed.

Tobias

-----------------
Mentor Graphics (Deutschland) GmbH, Arnulfstrasse 201, 80634 München Registergericht München HRB 106955, Geschäftsführer: Thomas Heurung, Frank Thürauf

[-- Attachment #2: committed.diff --]
[-- Type: text/x-patch, Size: 1501 bytes --]

commit 9faaa091e53db9cb4aee137bb4a6a9c6a18c7af5
Author: Tobias Burnus <tobias@codesourcery.com>
Date:   Mon May 31 16:38:05 2021 +0200

    gfortran.dg/gomp/depend-iterator-{1,2}.f90: Use dg-do compile
    
    'dg-do run' is pointless -1 due to dg-error. And it won't work except
    by chance for gomp tests; as -2 only has depend(out:), a 'dg-do compile'
    is sufficient.
    
    gcc/testsuite/ChangeLog:
    
            * gfortran.dg/gomp/depend-iterator-1.f90: Use dg-do compile.
            * gfortran.dg/gomp/depend-iterator-2.f90: Use dg-do compile.
---
 gcc/testsuite/gfortran.dg/gomp/depend-iterator-1.f90 | 2 +-
 gcc/testsuite/gfortran.dg/gomp/depend-iterator-2.f90 | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/gcc/testsuite/gfortran.dg/gomp/depend-iterator-1.f90 b/gcc/testsuite/gfortran.dg/gomp/depend-iterator-1.f90
index cad36aaf8b7..d852b95f1c2 100644
--- a/gcc/testsuite/gfortran.dg/gomp/depend-iterator-1.f90
+++ b/gcc/testsuite/gfortran.dg/gomp/depend-iterator-1.f90
@@ -1,4 +1,4 @@
-! { dg-do run }
+! { dg-do compile }
 
 module mymod
   implicit none (type, external)
diff --git a/gcc/testsuite/gfortran.dg/gomp/depend-iterator-2.f90 b/gcc/testsuite/gfortran.dg/gomp/depend-iterator-2.f90
index fa826a71097..21fc3272974 100644
--- a/gcc/testsuite/gfortran.dg/gomp/depend-iterator-2.f90
+++ b/gcc/testsuite/gfortran.dg/gomp/depend-iterator-2.f90
@@ -1,4 +1,4 @@
-! { dg-do run }
+! { dg-do compile }
 
 module mymod
   implicit none (type, external)

      parent reply	other threads:[~2021-05-31 14:43 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-04-27 13:36 Tobias Burnus
2021-04-28 13:41 ` Jakub Jelinek
2021-04-28 20:26   ` [Patch] Fortran/OpenMP: Fix var-list expr parsing with array/dt (was: [Patch] OpenMP: Add iterator support to Fortran's depend; add affinity clause) Tobias Burnus
2021-04-28 20:31     ` Jakub Jelinek
     [not found]   ` <3bfa68db-f904-a1cb-0d18-c1a17ecfc720@codesourcery.com>
     [not found]     ` <20210527082259.GM7746@tucnak>
2021-05-27 18:30       ` [Patch] OpenMP: Add iterator support to Fortran's depend; add affinity clause Tobias Burnus
2021-05-27 18:42         ` Jakub Jelinek
2021-05-27 19:58         ` Joseph Myers
2021-05-27 20:15           ` Jakub Jelinek
2021-05-27 22:59           ` Tobias Burnus
2021-05-29  8:03             ` Jakub Jelinek
2021-05-31  8:14               ` Christophe Lyon
2021-05-31  8:19                 ` Jakub Jelinek
2021-05-31 14:43               ` Tobias Burnus [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=3995781f-b673-df3c-12f8-a0635867ae43@codesourcery.com \
    --to=tobias@codesourcery.com \
    --cc=fortran@gcc.gnu.org \
    --cc=gcc-patches@gcc.gnu.org \
    --cc=jakub@redhat.com \
    /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).