https://gcc.gnu.org/g:6f759e76f00d70bf1d44a25767c73ec2de855452

commit 6f759e76f00d70bf1d44a25767c73ec2de855452
Author: Kwok Cheung Yeung <k...@codesourcery.com>
Date:   Tue Mar 24 09:36:42 2020 -0700

    Add XFAIL for libgomp.oacc-c-c++-common/data-firstprivate-1.c
    
    The firstprivate_int optimization changes the semantics of firstprivate
    in this test, so XFAIL it until the correct semantics for firstprivate
    have been decided (PR92036).
    
    2020-03-24  Kwok Cheung Yeung  <k...@codesourcery.com>
    
            libgomp/
            * testsuite/libgomp.oacc-c-c++-common/data-firstprivate-1.c: XFAIL
            execution test.

Diff:
---
 libgomp/ChangeLog.omp                                             | 5 +++++
 libgomp/testsuite/libgomp.oacc-c-c++-common/data-firstprivate-1.c | 6 ++++++
 2 files changed, 11 insertions(+)

diff --git a/libgomp/ChangeLog.omp b/libgomp/ChangeLog.omp
index 9389a0eefe3..cfc780519bb 100644
--- a/libgomp/ChangeLog.omp
+++ b/libgomp/ChangeLog.omp
@@ -1,3 +1,8 @@
+2020-03-24  Kwok Cheung Yeung  <k...@codesourcery.com>
+
+       * testsuite/libgomp.oacc-c-c++-common/data-firstprivate-1.c: XFAIL
+       execution test.
+
 2019-09-17  Julian Brown  <jul...@codesourcery.com>
            Kwok Cheung Yeung  <k...@codesourcery.com>
 
diff --git a/libgomp/testsuite/libgomp.oacc-c-c++-common/data-firstprivate-1.c 
b/libgomp/testsuite/libgomp.oacc-c-c++-common/data-firstprivate-1.c
index 8900a4e070d..4b88c53fac7 100644
--- a/libgomp/testsuite/libgomp.oacc-c-c++-common/data-firstprivate-1.c
+++ b/libgomp/testsuite/libgomp.oacc-c-c++-common/data-firstprivate-1.c
@@ -1,6 +1,12 @@
 /* Test behavior of 'firstprivate' lexically vs. dynamically nested inside a
    'data' region.  */
 
+/* The firstprivate_int optimization changes the semantics of firstprivate
+   in dynamically_nested_compute_2 to copy-by-value when not using shared
+   memory, leading to the behaviour suggested in PR92036 for this case.  */
+
+/* { dg-xfail-run-if "firstprivate_int" { *-*-* } { "-DACC_MEM_SHARED=0" } } */
+
 #include <stdlib.h>

Reply via email to