From patchwork Thu Sep 28 07:41:15 2017 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Tom de Vries X-Patchwork-Id: 819429 Return-Path: X-Original-To: incoming@patchwork.ozlabs.org Delivered-To: patchwork-incoming@bilbo.ozlabs.org Authentication-Results: ozlabs.org; spf=pass (mailfrom) smtp.mailfrom=gcc.gnu.org (client-ip=209.132.180.131; helo=sourceware.org; envelope-from=gcc-patches-return-463089-incoming=patchwork.ozlabs.org@gcc.gnu.org; receiver=) Authentication-Results: ozlabs.org; dkim=pass (1024-bit key; unprotected) header.d=gcc.gnu.org header.i=@gcc.gnu.org header.b="m5q2dJ6y"; dkim-atps=neutral Received: from sourceware.org (server1.sourceware.org [209.132.180.131]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ozlabs.org (Postfix) with ESMTPS id 3y2mpv2HNyz9t5x for ; Thu, 28 Sep 2017 17:41:38 +1000 (AEST) DomainKey-Signature: a=rsa-sha1; c=nofws; d=gcc.gnu.org; h=list-id :list-unsubscribe:list-archive:list-post:list-help:sender :subject:to:cc:references:from:message-id:date:mime-version :in-reply-to:content-type; q=dns; s=default; b=WqhM+mQRFOUkZVMsP SV7GyfyBBOsS13QYLSKlADj4jVLUrUGXrJZQjPYbDJmecg3kiZqeBA8QLAqV9C3d l7GKJKrn4lRCwsIP95XAwa5PF42D2CsCsPS/rR4d4iAuqZyMXWdHhI41m3ggQvGm ZFb+aW6Btn6Or3kzTPwTWOSDiw= DKIM-Signature: v=1; a=rsa-sha1; c=relaxed; d=gcc.gnu.org; h=list-id :list-unsubscribe:list-archive:list-post:list-help:sender :subject:to:cc:references:from:message-id:date:mime-version :in-reply-to:content-type; s=default; bh=1b/rmYJuHlksM4yvZvBYBW8 dKgw=; b=m5q2dJ6y672+T4IvdIs3oV95+XxQnrSirYGAZtCmu2MjUnyw87yd/N3 gnPXxDtzug2l7nXBptEN9BjjF7t+ULA+4/Lb3zaiFz2HGV5Lrsa1hxcSpLAX7hlM cgm/XDQQ74rN6QFo92M0qn8V9BqyaWvBsrYqMey/zpBObJawiVgU= Received: (qmail 116983 invoked by alias); 28 Sep 2017 07:41:29 -0000 Mailing-List: contact gcc-patches-help@gcc.gnu.org; run by ezmlm Precedence: bulk List-Id: List-Unsubscribe: List-Archive: List-Post: List-Help: Sender: gcc-patches-owner@gcc.gnu.org Delivered-To: mailing list gcc-patches@gcc.gnu.org Received: (qmail 115388 invoked by uid 89); 28 Sep 2017 07:41:27 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-24.6 required=5.0 tests=AWL, BAYES_00, GIT_PATCH_0, GIT_PATCH_1, GIT_PATCH_2, GIT_PATCH_3, RCVD_IN_DNSWL_NONE, SPF_PASS, URIBL_RED autolearn=ham version=3.3.2 spammy= X-HELO: relay1.mentorg.com Received: from relay1.mentorg.com (HELO relay1.mentorg.com) (192.94.38.131) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with ESMTP; Thu, 28 Sep 2017 07:41:24 +0000 Received: from nat-ies.mentorg.com ([192.94.31.2] helo=SVR-IES-MBX-04.mgc.mentorg.com) by relay1.mentorg.com with esmtps (TLSv1.2:ECDHE-RSA-AES256-SHA384:256) id 1dxTRi-0003dw-18 from Tom_deVries@mentor.com ; Thu, 28 Sep 2017 00:41:22 -0700 Received: from [127.0.0.1] (137.202.0.87) by SVR-IES-MBX-04.mgc.mentorg.com (139.181.222.4) with Microsoft SMTP Server (TLS) id 15.0.1263.5; Thu, 28 Sep 2017 08:41:18 +0100 Subject: [libgomp, testsuite] Remove superfluous -fopenmp from libgomp testcases To: Jakub Jelinek CC: , References: <878ux1jp2s.fsf@schwinge.name> <1383766943-8863-1-git-send-email-thomas@codesourcery.com> <1383766943-8863-2-git-send-email-thomas@codesourcery.com> <20131107081125.GO27813@tucnak.zalov.cz> From: Tom de Vries Message-ID: <6ff0d858-063b-2ba0-638f-9b5fb6bbf71c@mentor.com> Date: Thu, 28 Sep 2017 09:41:15 +0200 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.3.0 MIME-Version: 1.0 In-Reply-To: <20131107081125.GO27813@tucnak.zalov.cz> X-ClientProxiedBy: svr-ies-mbx-01.mgc.mentorg.com (139.181.222.1) To SVR-IES-MBX-04.mgc.mentorg.com (139.181.222.4) [ was: Re: [gomp4 2/9] libgomp: Prepare for testcases without -fopenmp. ] On 11/07/2013 09:11 AM, Jakub Jelinek wrote: > On Wed, Nov 06, 2013 at 08:42:16PM +0100, thomas@codesourcery.com wrote: >> From: Thomas Schwinge >> >> libgomp/ >> * testsuite/lib/libgomp.exp (libgomp_init): Don't add -fopenmp to >> ALWAYS_CFLAGS. >> * testsuite/libgomp.c++/c++.exp (ALWAYS_CFLAGS): Add -fopenmp. >> * testsuite/libgomp.c/c.exp (ALWAYS_CFLAGS): Likewise. >> * testsuite/libgomp.fortran/fortran.exp (ALWAYS_CFLAGS): Likewise. >> * testsuite/libgomp.graphite/graphite.exp (ALWAYS_CFLAGS): >> Likewise. > > Ok for trunk/gomp-4_0-branch. Following up on this, how about we drop the now superfluous -fopenmp in current test-cases? Tested on x86_64. Verified by analyzing libgomp.log that -fopenmp is still passed to test-cases as required. OK for trunk? Thanks, - Tom Remove superfluous -fopenmp from libgomp testcases 2017-09-16 Tom de Vries * testsuite/libgomp.c++/for-12.C: Remove superfluous -fopenmp option setting. * testsuite/libgomp.c++/pr69393.C: Same. * testsuite/libgomp.c++/taskloop-1.C: Same. * testsuite/libgomp.c++/taskloop-3.C: Same. * testsuite/libgomp.c++/taskloop-4.C: Same. * testsuite/libgomp.c/for-4.c: Same. * testsuite/libgomp.c/pr66199-3.c: Same. * testsuite/libgomp.c/pr66199-4.c: Same. * testsuite/libgomp.c/pr66199-6.c: Same. * testsuite/libgomp.c/taskloop-1.c: Same. * testsuite/libgomp.c/taskloop-3.c: Same. * testsuite/libgomp.c/taskloop-4.c: Same. * testsuite/libgomp.fortran/aligned1.f03: Same. * testsuite/libgomp.fortran/condinc1.f: Same. * testsuite/libgomp.fortran/condinc3.f90: Same. * testsuite/libgomp.fortran/crayptr1.f90: Same. * testsuite/libgomp.fortran/crayptr2.f90: Same. * testsuite/libgomp.fortran/crayptr3.f90: Same. * testsuite/libgomp.fortran/omp_cond1.f: Same. * testsuite/libgomp.fortran/omp_cond3.F90: Same. * testsuite/libgomp.fortran/pr66199-1.f90: Same. * testsuite/libgomp.fortran/pr66199-2.f90: Same. * testsuite/libgomp.fortran/recursion1.f90: Same. * testsuite/libgomp.fortran/target2.f90: Same. * testsuite/libgomp.fortran/target5.f90: Same. * testsuite/libgomp.fortran/task3.f90: Same. --- libgomp/testsuite/libgomp.c++/for-12.C | 2 -- libgomp/testsuite/libgomp.c++/pr69393.C | 2 +- libgomp/testsuite/libgomp.c++/taskloop-1.C | 2 +- libgomp/testsuite/libgomp.c++/taskloop-3.C | 2 +- libgomp/testsuite/libgomp.c++/taskloop-4.C | 2 +- libgomp/testsuite/libgomp.c/for-4.c | 2 +- libgomp/testsuite/libgomp.c/pr66199-3.c | 2 +- libgomp/testsuite/libgomp.c/pr66199-4.c | 2 +- libgomp/testsuite/libgomp.c/pr66199-6.c | 2 +- libgomp/testsuite/libgomp.c/taskloop-1.c | 2 +- libgomp/testsuite/libgomp.c/taskloop-3.c | 2 +- libgomp/testsuite/libgomp.c/taskloop-4.c | 2 +- libgomp/testsuite/libgomp.fortran/aligned1.f03 | 2 +- libgomp/testsuite/libgomp.fortran/condinc1.f | 1 - libgomp/testsuite/libgomp.fortran/condinc3.f90 | 1 - libgomp/testsuite/libgomp.fortran/crayptr1.f90 | 2 +- libgomp/testsuite/libgomp.fortran/crayptr2.f90 | 2 +- libgomp/testsuite/libgomp.fortran/crayptr3.f90 | 2 +- libgomp/testsuite/libgomp.fortran/omp_cond1.f | 1 - libgomp/testsuite/libgomp.fortran/omp_cond3.F90 | 1 - libgomp/testsuite/libgomp.fortran/pr66199-1.f90 | 2 +- libgomp/testsuite/libgomp.fortran/pr66199-2.f90 | 2 +- libgomp/testsuite/libgomp.fortran/recursion1.f90 | 2 +- libgomp/testsuite/libgomp.fortran/target2.f90 | 2 +- libgomp/testsuite/libgomp.fortran/target5.f90 | 1 - libgomp/testsuite/libgomp.fortran/task3.f90 | 1 - 26 files changed, 19 insertions(+), 27 deletions(-) diff --git a/libgomp/testsuite/libgomp.c++/for-12.C b/libgomp/testsuite/libgomp.c++/for-12.C index ea32192..295b12f 100644 --- a/libgomp/testsuite/libgomp.c++/for-12.C +++ b/libgomp/testsuite/libgomp.c++/for-12.C @@ -1,5 +1,3 @@ -/* { dg-options "-fopenmp" } */ - extern "C" void abort (void); #define M(x, y, z) O(x, y, z) diff --git a/libgomp/testsuite/libgomp.c++/pr69393.C b/libgomp/testsuite/libgomp.c++/pr69393.C index e3f0de1..02605e0 100644 --- a/libgomp/testsuite/libgomp.c++/pr69393.C +++ b/libgomp/testsuite/libgomp.c++/pr69393.C @@ -1,6 +1,6 @@ // { dg-do run } // { dg-require-effective-target lto } -// { dg-options "-flto -g -fopenmp" } +// { dg-options "-flto -g" } int e = 5; diff --git a/libgomp/testsuite/libgomp.c++/taskloop-1.C b/libgomp/testsuite/libgomp.c++/taskloop-1.C index 66f8e0b..7fc6e46 100644 --- a/libgomp/testsuite/libgomp.c++/taskloop-1.C +++ b/libgomp/testsuite/libgomp.c++/taskloop-1.C @@ -1,4 +1,4 @@ // { dg-do run } -// { dg-options "-O2 -fopenmp" } +// { dg-options "-O2" } #include "../libgomp.c/taskloop-1.c" diff --git a/libgomp/testsuite/libgomp.c++/taskloop-3.C b/libgomp/testsuite/libgomp.c++/taskloop-3.C index bfd793c..c08a045 100644 --- a/libgomp/testsuite/libgomp.c++/taskloop-3.C +++ b/libgomp/testsuite/libgomp.c++/taskloop-3.C @@ -1,4 +1,4 @@ // { dg-do run } -// { dg-options "-O2 -fopenmp" } +// { dg-options "-O2" } #include "../libgomp.c/taskloop-3.c" diff --git a/libgomp/testsuite/libgomp.c++/taskloop-4.C b/libgomp/testsuite/libgomp.c++/taskloop-4.C index 937cfcc..3783717 100644 --- a/libgomp/testsuite/libgomp.c++/taskloop-4.C +++ b/libgomp/testsuite/libgomp.c++/taskloop-4.C @@ -1,4 +1,4 @@ // { dg-do run } -// { dg-options "-O2 -fopenmp" } +// { dg-options "-O2" } #include "../libgomp.c/taskloop-4.c" diff --git a/libgomp/testsuite/libgomp.c/for-4.c b/libgomp/testsuite/libgomp.c/for-4.c index ef5465e..14f900b 100644 --- a/libgomp/testsuite/libgomp.c/for-4.c +++ b/libgomp/testsuite/libgomp.c/for-4.c @@ -1,4 +1,4 @@ -/* { dg-options "-std=gnu99 -fopenmp" } */ +/* { dg-options "-std=gnu99" } */ extern void abort (void); diff --git a/libgomp/testsuite/libgomp.c/pr66199-3.c b/libgomp/testsuite/libgomp.c/pr66199-3.c index fe0ccb4..ffe2858 100644 --- a/libgomp/testsuite/libgomp.c/pr66199-3.c +++ b/libgomp/testsuite/libgomp.c/pr66199-3.c @@ -1,6 +1,6 @@ /* PR middle-end/66199 */ /* { dg-do run } */ -/* { dg-options "-O2 -fopenmp" } */ +/* { dg-options "-O2" } */ int u[1024], v[1024], w[1024]; diff --git a/libgomp/testsuite/libgomp.c/pr66199-4.c b/libgomp/testsuite/libgomp.c/pr66199-4.c index a9b1bb8..6ffa2e5 100644 --- a/libgomp/testsuite/libgomp.c/pr66199-4.c +++ b/libgomp/testsuite/libgomp.c/pr66199-4.c @@ -1,6 +1,6 @@ /* PR middle-end/66199 */ /* { dg-do run } */ -/* { dg-options "-O2 -fopenmp" } */ +/* { dg-options "-O2" } */ #pragma omp declare target int u[1024], v[1024], w[1024]; diff --git a/libgomp/testsuite/libgomp.c/pr66199-6.c b/libgomp/testsuite/libgomp.c/pr66199-6.c index 6790f7d..14f8610 100644 --- a/libgomp/testsuite/libgomp.c/pr66199-6.c +++ b/libgomp/testsuite/libgomp.c/pr66199-6.c @@ -1,6 +1,6 @@ /* PR middle-end/66199 */ /* { dg-do run } */ -/* { dg-options "-O2 -fopenmp" } */ +/* { dg-options "-O2" } */ #pragma omp declare target int u[1024], v[1024], w[1024]; diff --git a/libgomp/testsuite/libgomp.c/taskloop-1.c b/libgomp/testsuite/libgomp.c/taskloop-1.c index 21551f2..edc7058 100644 --- a/libgomp/testsuite/libgomp.c/taskloop-1.c +++ b/libgomp/testsuite/libgomp.c/taskloop-1.c @@ -1,5 +1,5 @@ /* { dg-do run } */ -/* { dg-options "-O2 -fopenmp -std=c99" } */ +/* { dg-options "-O2 -std=c99" } */ int q, r, e; diff --git a/libgomp/testsuite/libgomp.c/taskloop-3.c b/libgomp/testsuite/libgomp.c/taskloop-3.c index 5356d7f..9c8c49c 100644 --- a/libgomp/testsuite/libgomp.c/taskloop-3.c +++ b/libgomp/testsuite/libgomp.c/taskloop-3.c @@ -1,5 +1,5 @@ /* { dg-do run } */ -/* { dg-options "-O2 -fopenmp -std=c99" } */ +/* { dg-options "-O2 -std=c99" } */ int g; int a[1024]; diff --git a/libgomp/testsuite/libgomp.c/taskloop-4.c b/libgomp/testsuite/libgomp.c/taskloop-4.c index a69be19..4ac1b5a 100644 --- a/libgomp/testsuite/libgomp.c/taskloop-4.c +++ b/libgomp/testsuite/libgomp.c/taskloop-4.c @@ -1,5 +1,5 @@ /* { dg-do run } */ -/* { dg-options "-O2 -fopenmp" } */ +/* { dg-options "-O2" } */ int u[64], v; diff --git a/libgomp/testsuite/libgomp.fortran/aligned1.f03 b/libgomp/testsuite/libgomp.fortran/aligned1.f03 index 67a9ab4..2db03e1 100644 --- a/libgomp/testsuite/libgomp.fortran/aligned1.f03 +++ b/libgomp/testsuite/libgomp.fortran/aligned1.f03 @@ -1,5 +1,5 @@ ! { dg-do run } -! { dg-options "-fopenmp -fcray-pointer" } +! { dg-options "-fcray-pointer" } use iso_c_binding, only : c_ptr, c_ptrdiff_t, c_loc interface diff --git a/libgomp/testsuite/libgomp.fortran/condinc1.f b/libgomp/testsuite/libgomp.fortran/condinc1.f index d94fe8d..6d05635 100644 --- a/libgomp/testsuite/libgomp.fortran/condinc1.f +++ b/libgomp/testsuite/libgomp.fortran/condinc1.f @@ -1,4 +1,3 @@ -! { dg-options "-fopenmp" } program condinc1 logical l l = .false. diff --git a/libgomp/testsuite/libgomp.fortran/condinc3.f90 b/libgomp/testsuite/libgomp.fortran/condinc3.f90 index 16b937a..7c24303 100644 --- a/libgomp/testsuite/libgomp.fortran/condinc3.f90 +++ b/libgomp/testsuite/libgomp.fortran/condinc3.f90 @@ -1,4 +1,3 @@ - ! { dg-options "-fopenmp" } program condinc3 logical l l = .false. diff --git a/libgomp/testsuite/libgomp.fortran/crayptr1.f90 b/libgomp/testsuite/libgomp.fortran/crayptr1.f90 index 57c59f7..fb9495e 100644 --- a/libgomp/testsuite/libgomp.fortran/crayptr1.f90 +++ b/libgomp/testsuite/libgomp.fortran/crayptr1.f90 @@ -1,5 +1,5 @@ ! { dg-do run } -! { dg-options "-fopenmp -fcray-pointer" } +! { dg-options "-fcray-pointer" } use omp_lib integer :: a, b, c, p diff --git a/libgomp/testsuite/libgomp.fortran/crayptr2.f90 b/libgomp/testsuite/libgomp.fortran/crayptr2.f90 index c88cc7a..7e69b87 100644 --- a/libgomp/testsuite/libgomp.fortran/crayptr2.f90 +++ b/libgomp/testsuite/libgomp.fortran/crayptr2.f90 @@ -1,5 +1,5 @@ ! { dg-do run } -! { dg-options "-fopenmp -fcray-pointer" } +! { dg-options "-fcray-pointer" } ! { dg-require-effective-target tls_runtime } use omp_lib diff --git a/libgomp/testsuite/libgomp.fortran/crayptr3.f90 b/libgomp/testsuite/libgomp.fortran/crayptr3.f90 index 9777c6b..5b3de65 100644 --- a/libgomp/testsuite/libgomp.fortran/crayptr3.f90 +++ b/libgomp/testsuite/libgomp.fortran/crayptr3.f90 @@ -1,5 +1,5 @@ ! { dg-do run } -! { dg-options "-fopenmp -fcray-pointer" } +! { dg-options "-fcray-pointer" } use omp_lib integer :: a, b, c, i, p diff --git a/libgomp/testsuite/libgomp.fortran/omp_cond1.f b/libgomp/testsuite/libgomp.fortran/omp_cond1.f index b557d90..f51e868 100644 --- a/libgomp/testsuite/libgomp.fortran/omp_cond1.f +++ b/libgomp/testsuite/libgomp.fortran/omp_cond1.f @@ -1,5 +1,4 @@ C Test conditional compilation in fixed form if -fopenmp -! { dg-options "-fopenmp" } 10 foo = 2 &56 if (foo.ne.256) call abort diff --git a/libgomp/testsuite/libgomp.fortran/omp_cond3.F90 b/libgomp/testsuite/libgomp.fortran/omp_cond3.F90 index 6c4e36e..fe8c783 100644 --- a/libgomp/testsuite/libgomp.fortran/omp_cond3.F90 +++ b/libgomp/testsuite/libgomp.fortran/omp_cond3.F90 @@ -1,5 +1,4 @@ ! Test conditional compilation in free form if -fopenmp -! { dg-options "-fopenmp" } 10 foo = 2& &56 if (foo.ne.256) call abort diff --git a/libgomp/testsuite/libgomp.fortran/pr66199-1.f90 b/libgomp/testsuite/libgomp.fortran/pr66199-1.f90 index 0cd232f..55ad627 100644 --- a/libgomp/testsuite/libgomp.fortran/pr66199-1.f90 +++ b/libgomp/testsuite/libgomp.fortran/pr66199-1.f90 @@ -1,6 +1,6 @@ ! PR middle-end/66199 ! { dg-do run } -! { dg-options "-O2 -fopenmp" } +! { dg-options "-O2" } integer :: u(1024), v(1024), w(1024), a, b, c, d, e, a1, b1, a2, b2, d1, d2 a = 1 diff --git a/libgomp/testsuite/libgomp.fortran/pr66199-2.f90 b/libgomp/testsuite/libgomp.fortran/pr66199-2.f90 index e17ab96..0cc0fa5 100644 --- a/libgomp/testsuite/libgomp.fortran/pr66199-2.f90 +++ b/libgomp/testsuite/libgomp.fortran/pr66199-2.f90 @@ -1,6 +1,6 @@ ! PR middle-end/66199 ! { dg-do run } -! { dg-options "-O2 -fopenmp" } +! { dg-options "-O2" } integer :: u(1024), v(1024), w(1024), a, b, c, d, e, a1, b1, a2, b2, d1, d2 a = 1 diff --git a/libgomp/testsuite/libgomp.fortran/recursion1.f90 b/libgomp/testsuite/libgomp.fortran/recursion1.f90 index 35cb878..0cae261 100644 --- a/libgomp/testsuite/libgomp.fortran/recursion1.f90 +++ b/libgomp/testsuite/libgomp.fortran/recursion1.f90 @@ -1,5 +1,5 @@ ! { dg-do run } -! { dg-options "-fopenmp -fcheck=recursion" } +! { dg-options "-fcheck=recursion" } ! ! PR 42517: Bogus runtime error with -fopenmp -fcheck=recursion ! diff --git a/libgomp/testsuite/libgomp.fortran/target2.f90 b/libgomp/testsuite/libgomp.fortran/target2.f90 index 42f704f..40b7b0b 100644 --- a/libgomp/testsuite/libgomp.fortran/target2.f90 +++ b/libgomp/testsuite/libgomp.fortran/target2.f90 @@ -1,5 +1,5 @@ ! { dg-do run } -! { dg-options "-fopenmp -ffree-line-length-160" } +! { dg-options "-ffree-line-length-160" } module target2 contains diff --git a/libgomp/testsuite/libgomp.fortran/target5.f90 b/libgomp/testsuite/libgomp.fortran/target5.f90 index c46faf2..4aad88e 100644 --- a/libgomp/testsuite/libgomp.fortran/target5.f90 +++ b/libgomp/testsuite/libgomp.fortran/target5.f90 @@ -1,5 +1,4 @@ ! { dg-do compile } -! { dg-options "-fopenmp" } integer :: r r = 0 diff --git a/libgomp/testsuite/libgomp.fortran/task3.f90 b/libgomp/testsuite/libgomp.fortran/task3.f90 index 30ff980..83b0e9f 100644 --- a/libgomp/testsuite/libgomp.fortran/task3.f90 +++ b/libgomp/testsuite/libgomp.fortran/task3.f90 @@ -1,5 +1,4 @@ ! { dg-do run } -! { dg-options "-fopenmp" } ! ! PR fortran/47886 !