Re: [Patch] OpenMP/C++: Avoid ICE for BIND_EXPR with empty BIND_EXPR_BLOCK [PR120413]

[Tobias Burnus]

Jakub Jelinek wrote:
There is also BIND_EXPR_VARS, dunno if that should be walked instead or
in addition.

The usage is to ensure that variables are mapped with lambdas (→ 
closure_vars_accessed.add (…)) but not if they are local variables (→ 
data->local_decls.add (var)).

The 'closure_vars_accessed.add(…)' code currently only triggers for 
libgomp.c++/target-lambda-1.C.

I will leave the PR open for some more testing and test-case adding.

However, as an ICE is bad, I now committed the patch with the suggested 
change:

Anyway, from formatting POV, it would be nicer to do
...
       if (tree block = BIND_EXPR_BLOCK (t))

Done so - and committed as r16-881-g45b849d05b733a.

Thanks,

Tobias