On 18 October 2019 17:08:54 CEST, Chung-Lin Tang <chunglin_t...@mentor.com> 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 <ce...@codesourcery.com> > Chung-Lin Tang <clt...@codesourcery.com> > > 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 <clt...@codesourcery.com> > James Norris <jnor...@codesourcery.com> > > 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.