On Wed, Mar 07, 2018 at 02:29:48PM +0100, Richard Biener wrote:
> On Wed, 7 Mar 2018, Jakub Jelinek wrote:
>
> > On Wed, Mar 07, 2018 at 02:20:26PM +0100, Tom de Vries wrote:
> > > Fix ICE for static vars in offloaded functions
> > >
> > > 2018-03-06 Tom de Vries <t...@codesourcery.com>
> > >
> > > PR lto/84592
> > > * varpool.c (varpool_node::get_create): Mark static variables in
> > > offloaded functions as offloadable.
> > >
> > > * testsuite/libgomp.c/pr84592-2.c: New test.
> > > * testsuite/libgomp.c/pr84592.c: New test.
> > > * testsuite/libgomp.oacc-c-c++-common/pr84592-3.c: New test.
> >
> > Ok, thanks
>
> + bool in_offload_func
> + = (cfun
> + && TREE_STATIC (decl)
> + && (lookup_attribute ("omp target entr
>
> I think you want to use decl_function_context (decl) here,
> not rely on magic cfun being set. The whole varpool.c file
> doesn't mention cfun yet and you shoudln't either.
Oops, sure, thanks for catching it.
Jakub
