[gcc/devel/omp/gcc-9] [Fortran] Disable front-end optimization for OpenACC atomic (PR93462)

Tobias Burnus burnus@gcc.gnu.org
Thu Mar 5 14:45:00 GMT 2020


https://gcc.gnu.org/g:e5446f2201d93fc9adc913ed320aa70437ff4235

commit e5446f2201d93fc9adc913ed320aa70437ff4235
Author: Tobias Burnus <tobias@codesourcery.com>
Date:   Mon Feb 3 12:09:46 2020 +0100

    [Fortran] Disable front-end optimization for OpenACC atomic (PR93462)
    
    	Backported from mainline
    	2020-01-31  Tobias Burnus  <tobias@codesourcery.com>
    
    	PR fortran/93462
    	* frontend-passes.c (gfc_code_walker): For EXEC_OACC_ATOMIC, set
    	in_omp_atomic to true prevent front-end optimization.
    
    	PR fortran/93462
    	* gfortran.dg/goacc/atomic-1.f90: New.

Diff:
---
 gcc/fortran/ChangeLog                        |  9 +++++++++
 gcc/fortran/frontend-passes.c                |  1 +
 gcc/testsuite/ChangeLog                      |  8 ++++++++
 gcc/testsuite/gfortran.dg/goacc/atomic-1.f90 | 17 +++++++++++++++++
 4 files changed, 35 insertions(+)

diff --git a/gcc/fortran/ChangeLog b/gcc/fortran/ChangeLog
index ec87cc9..ca80c1b 100644
--- a/gcc/fortran/ChangeLog
+++ b/gcc/fortran/ChangeLog
@@ -1,6 +1,15 @@
 2020-02-03  Tobias Burnus  <tobias@codesourcery.com>
 
 	Backported from mainline
+	2020-01-31  Tobias Burnus  <tobias@codesourcery.com>
+
+	PR fortran/93462
+	* frontend-passes.c (gfc_code_walker): For EXEC_OACC_ATOMIC, set
+	in_omp_atomic to true prevent front-end optimization.
+
+2020-02-03  Tobias Burnus  <tobias@codesourcery.com>
+
+	Backported from mainline
 	2020-01-21  Tobias Burnus  <tobias@codesourcery.com>
 
 	PR fortran/93309
diff --git a/gcc/fortran/frontend-passes.c b/gcc/fortran/frontend-passes.c
index 4bb6cbb..a6e710b 100644
--- a/gcc/fortran/frontend-passes.c
+++ b/gcc/fortran/frontend-passes.c
@@ -5260,6 +5260,7 @@ gfc_code_walker (gfc_code **c, walk_code_fn_t codefn, walk_expr_fn_t exprfn,
 	      WALK_SUBEXPR (co->ext.dt->extra_comma);
 	      break;
 
+	    case EXEC_OACC_ATOMIC:
 	    case EXEC_OMP_ATOMIC:
 	      in_omp_atomic = true;
 	      break;
diff --git a/gcc/testsuite/ChangeLog b/gcc/testsuite/ChangeLog
index 0f8e759..e3610fd 100644
--- a/gcc/testsuite/ChangeLog
+++ b/gcc/testsuite/ChangeLog
@@ -1,6 +1,14 @@
 2020-02-03  Tobias Burnus  <tobias@codesourcery.com>
 
 	Backported from mainline
+	2020-01-31  Tobias Burnus  <tobias@codesourcery.com>
+
+	PR fortran/93462
+	* gfortran.dg/goacc/atomic-1.f90: New.
+
+2020-02-03  Tobias Burnus  <tobias@codesourcery.com>
+
+	Backported from mainline
 	2020-01-21  Tobias Burnus  <tobias@codesourcery.com>
 
 	PR fortran/93309
diff --git a/gcc/testsuite/gfortran.dg/goacc/atomic-1.f90 b/gcc/testsuite/gfortran.dg/goacc/atomic-1.f90
new file mode 100644
index 0000000..579f049
--- /dev/null
+++ b/gcc/testsuite/gfortran.dg/goacc/atomic-1.f90
@@ -0,0 +1,17 @@
+! { dg-do compile }
+!
+! PR fortran/93462
+!
+! Contributed by G. Steinmetz
+!
+program p
+   integer :: n = 1
+   integer :: a
+!$acc atomic write
+   a = f(n) - f(n)
+contains
+   integer function f(x)
+      integer, intent(in) :: x
+      f = x
+   end
+end



More information about the Gcc-cvs mailing list