On 18 October 2019 17:08:54 CEST, Chung-Lin Tang <[email protected]> wrote:
>Also, I've added a new libgomp.oacc-fortran/deviceptr-2.f90 testcase >that >actually copies out and verifies the deviceptr computation. In testcases please do not 'call abort' which is nonstandard but use 'stop N' which is standard, ideally with different stop integers so one can see easily which test failed. We went through all of the testsuite a while ago to remove the nonstandard abort, FYI. TIA, >Is this okay for trunk now? > >Thanks, >Chung-Lin > >2019-10-18 Cesar Philippidis <[email protected]> > Chung-Lin Tang <[email protected]> > > gcc/fortran/ > * trans-openmp.c (gfc_omp_finish_clause): Don't create pointer data > mappings for deviceptr clauses. > (gfc_trans_omp_clauses): Likewise. > > gcc/ > * gimplify.c (enum gimplify_omp_var_data): Add GOVD_DEVICETPR. > (omp_notice_variable): Add GOVD_DEVICEPTR attribute when appropriate. > (gimplify_scan_omp_clauses): Likewise. > (gimplify_adjust_omp_clauses_1): Set GOMP_MAP_FORCE_DEVICEPTR for > implicit deviceptr mappings. > gcc/testsuite/ > * c-c++-common/goacc/deviceptr-4.c: Update expected data mapping. > >2019-10-18 Chung-Lin Tang <[email protected]> > James Norris <[email protected]> > > libgomp/ > * oacc-parallel.c (handle_ftn_pointers): Delete function. > (GOACC_parallel_keyed): Remove call to handle_ftn_pointers. > * testsuite/libgomp.oacc-fortran/deviceptr-1.f90: New test. > * testsuite/libgomp.oacc-fortran/deviceptr-2.f90: New test.
