This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [Patch] tree-parloops.c (eliminate_local_variables): Add braces to suppress warnings
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)
>>>
>>
>
>