On Tue, Apr 10, 2012 at 3:06 PM, JonY <jo...@users.sourceforge.net> wrote: > On 4/10/2012 20:37, Richard Guenther wrote: >> On Tue, Apr 10, 2012 at 2:15 PM, JonY wrote: >>> Hi, >>> >>> Patch OK? >> >> What kind of warning? >> > > Oops, I forgot to mention gcc was complaining about missing braces.
Not for me. And I fail to see why it should. Richard. >>> ChangeLog: >>> tree-parloops.c (eliminate_local_variables): Add braces to suppress >>> warnings. >>> >>> Index: tree-parloops.c >>> =================================================================== >>> --- tree-parloops.c (revision 186243) >>> +++ tree-parloops.c (working copy) >>> @@ -723,13 +723,15 @@ >>> FOR_EACH_VEC_ELT (basic_block, body, i, bb) >>> if (bb != entry_bb && bb != exit_bb) >>> for (gsi = gsi_start_bb (bb); !gsi_end_p (gsi); gsi_next (&gsi)) >>> - if (is_gimple_debug (gsi_stmt (gsi))) >>> - { >>> - if (gimple_debug_bind_p (gsi_stmt (gsi))) >>> - has_debug_stmt = true; >>> - } >>> - else >>> - eliminate_local_variables_stmt (entry, &gsi, decl_address); >>> + { >>> + if (is_gimple_debug (gsi_stmt (gsi))) >>> + { >>> + if (gimple_debug_bind_p (gsi_stmt (gsi))) >>> + has_debug_stmt = true; >>> + } >>> + else >>> + eliminate_local_variables_stmt (entry, &gsi, decl_address); >>> + } >>> >>> if (has_debug_stmt) >>> FOR_EACH_VEC_ELT (basic_block, body, i, bb) >>> >> > >