On Mon, Sep 10, 2018 at 4:05 AM, Julian Brown <jul...@codesourcery.com> wrote: > This patch (by Cesar) changes the way C++ references are mapped in > OpenACC regions, fixing an ICE in the non-scalar-data.C testcase. > > Post-patch, references are mapped like this (from the omplower dump): > > map(force_present:*x [len: 4]) map(firstprivate ref:x [pointer assign, bias: > 0]) > > Tested with offloading to NVPTX and bootstrapped. OK for trunk? > > Thanks, > > Julian > > ChangeLog > > 2018-09-09 Cesar Philippidis <ce...@codesourcery.com> > Julian Brown <jul...@codesourcery.com> > > PR middle-end/86336 > > (gimplify_adjust_omp_clauses_1): Update handling of mapping of C++ > references.
How is reference handling specified differently between OpenMP and OpenACC? It seems strange for them to differ. In any case, you shouldn't need to check lang_GNU_CXX since we're already calling the langhook. Jason