From 9f1b833bc3f28a3000f2ce8c076a1930f44bcda9 Mon Sep 17 00:00:00 2001 From: Tom de Vries Date: Fri, 8 Apr 2016 09:41:16 +0000 Subject: [PATCH] Remove incorrect warning for parallel implicit firstprivate clause 2016-04-08 Tom de Vries * omp-low.c (lower_omp_target): Set TREE_NO_WARNING for oacc implicit firstprivate clause. * c-c++-common/goacc/uninit-firstprivate-clause.c: New test. * gfortran.dg/goacc/uninit-firstprivate-clause.f95: New test. From-SVN: r234826 --- gcc/ChangeLog | 5 ++++ gcc/omp-low.c | 7 +++++- gcc/testsuite/ChangeLog | 5 ++++ .../goacc/uninit-firstprivate-clause.c | 25 +++++++++++++++++++ .../goacc/uninit-firstprivate-clause.f95 | 18 +++++++++++++ 5 files changed, 59 insertions(+), 1 deletion(-) create mode 100644 gcc/testsuite/c-c++-common/goacc/uninit-firstprivate-clause.c create mode 100644 gcc/testsuite/gfortran.dg/goacc/uninit-firstprivate-clause.f95 diff --git a/gcc/ChangeLog b/gcc/ChangeLog index 7fe1c42eda7a..158f4cc7a9c4 100644 --- a/gcc/ChangeLog +++ b/gcc/ChangeLog @@ -1,3 +1,8 @@ +2016-04-08 Tom de Vries + + * omp-low.c (lower_omp_target): Set TREE_NO_WARNING for oacc + implicit firstprivate clause. + 2016-04-08 Kyrylo Tkachov PR target/70566 diff --git a/gcc/omp-low.c b/gcc/omp-low.c index 979926d332a7..710519442df1 100644 --- a/gcc/omp-low.c +++ b/gcc/omp-low.c @@ -16077,7 +16077,12 @@ lower_omp_target (gimple_stmt_iterator *gsi_p, omp_context *ctx) { gcc_assert (is_gimple_omp_oacc (ctx->stmt)); if (!is_reference (var)) - var = build_fold_addr_expr (var); + { + if (is_gimple_reg (var) + && OMP_CLAUSE_FIRSTPRIVATE_IMPLICIT (c)) + TREE_NO_WARNING (var) = 1; + var = build_fold_addr_expr (var); + } else talign = TYPE_ALIGN_UNIT (TREE_TYPE (TREE_TYPE (ovar))); gimplify_assign (x, var, &ilist); diff --git a/gcc/testsuite/ChangeLog b/gcc/testsuite/ChangeLog index 2668b0d92af4..6ca26101a927 100644 --- a/gcc/testsuite/ChangeLog +++ b/gcc/testsuite/ChangeLog @@ -1,3 +1,8 @@ +2016-04-08 Tom de Vries + + * c-c++-common/goacc/uninit-firstprivate-clause.c: New test. + * gfortran.dg/goacc/uninit-firstprivate-clause.f95: New test. + 2016-04-08 Kyrylo Tkachov PR target/70566 diff --git a/gcc/testsuite/c-c++-common/goacc/uninit-firstprivate-clause.c b/gcc/testsuite/c-c++-common/goacc/uninit-firstprivate-clause.c new file mode 100644 index 000000000000..2584033a8c56 --- /dev/null +++ b/gcc/testsuite/c-c++-common/goacc/uninit-firstprivate-clause.c @@ -0,0 +1,25 @@ +/* { dg-do compile } */ +/* { dg-additional-options "-Wuninitialized" } */ + +void +foo (void) +{ + int i; + +#pragma acc parallel + { + i = 1; + } +} + + +void +foo2 (void) +{ + int i; + +#pragma acc parallel firstprivate (i) /* { dg-warning "is used uninitialized in this function" } */ + { + i = 1; + } +} diff --git a/gcc/testsuite/gfortran.dg/goacc/uninit-firstprivate-clause.f95 b/gcc/testsuite/gfortran.dg/goacc/uninit-firstprivate-clause.f95 new file mode 100644 index 000000000000..14d960a1f385 --- /dev/null +++ b/gcc/testsuite/gfortran.dg/goacc/uninit-firstprivate-clause.f95 @@ -0,0 +1,18 @@ +! { dg-do compile } +! { dg-additional-options "-Wuninitialized" } + +subroutine test + INTEGER :: i + + !$acc parallel + i = 1 + !$acc end parallel +end subroutine test + +subroutine test2 + INTEGER :: i + + !$acc parallel firstprivate (i) ! { dg-warning "is used uninitialized in this function" } + i = 1 + !$acc end parallel +end subroutine test2