[Patch] tree-parloops.c (eliminate_local_variables): Add braces to suppress warnings

Richard Guenther richard.guenther@gmail.com
Tue Apr 10 14:38:00 GMT 2012


On Tue, Apr 10, 2012 at 3:06 PM, JonY <jon_y@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)
>>>
>>
>
>



More information about the Gcc-patches mailing list