On Wed, Jun 08, 2022 at 09:54:07AM +0200, Tobias Burnus wrote:
> The OpenMP requires directive may only be placed in the specification part of
> a program unit (except it happens via the USE of a module).
>
> But the target directive ancestor-requires-'reverse_offload' only checked
> the current namespace.
>
> OK for mainline?
>
> Tobias
> -----------------
> Siemens Electronic Design Automation GmbH; Anschrift: Arnulfstraße 201, 80634
> München; Gesellschaft mit beschränkter Haftung; Geschäftsführer: Thomas
> Heurung, Frank Thürauf; Sitz der Gesellschaft: München; Registergericht
> München, HRB 106955
> OpenMP: Fortran - fix ancestor's requires reverse_offload check
>
> gcc/fortran/
>
> * openmp.cc (gfc_match_omp_clauses): Check also parent namespace
> for 'requires reverse_offload'.
>
> gcc/testsuite/
>
> * gfortran.dg/gomp/target-device-ancestor-5.f90: New test.
LGTM, thanks.
Jakub