Hi, On 09.12.19 16:58, Harwath, Frederik wrote: > Tobias has recently fixed a problem with the column information in gfortran > locations > [...] > I have tested the patch manually by adapting the validity check for nested > OpenACC reductions (see omp-low.c) > to include the location of clauses in warnings instead of the location of the > loop to which the clause belongs. > I can add a regression test based on this later on after adapting the code in > omp-low.c.
here are patches adding the promised test for Fortran and a corresponding test for C. Is it ok to include them in trunk? Best regards, Frederik Frederik Harwath (2): Use clause locations in OpenACC nested reduction warnings Add tests to verify OpenACC clause locations gcc/omp-low.c | 2 +- gcc/testsuite/gcc.dg/goacc/clause-locations.c | 17 +++++++++++++++++ .../gfortran.dg/goacc/clause-locations.f90 | 18 ++++++++++++++++++ 3 files changed, 36 insertions(+), 1 deletion(-) create mode 100644 gcc/testsuite/gcc.dg/goacc/clause-locations.c create mode 100644 gcc/testsuite/gfortran.dg/goacc/clause-locations.f90 -- 2.17.1
