On Thu, Oct 17, 2013 at 11:52:21AM -0400, Andrew MacLeod wrote: > On 10/17/2013 11:15 AM, Jakub Jelinek wrote: > >>*************** struct omp_for_data > >>*** 135,141 **** > >> static splay_tree all_contexts; > >> static int taskreg_nesting_level; > >> static int target_nesting_level; > >>! struct omp_region *root_omp_region; > >> static bitmap task_shared_vars; > >> static void scan_omp (gimple_seq *, omp_context *); > >>--- 175,181 ---- > >> static splay_tree all_contexts; > >> static int taskreg_nesting_level; > >> static int target_nesting_level; > >>! static struct omp_region *root_omp_region; > >> static bitmap task_shared_vars; > >> static void scan_omp (gimple_seq *, omp_context *); > >Why? > It should be static now since it is no longer exported outside the > file... and can't be now that struct omp_region is declared in > omp-low.c
Ah, just misread your change, thought you are adding struct keyword while you are actually adding static. Sorry. Jakub