On Tue, May 31, 2016 at 9:40 AM, Marek Polacek <pola...@redhat.com> wrote:
> On Fri, May 27, 2016 at 10:41:51AM -0400, Jason Merrill wrote:
>> On 05/26/2016 02:44 PM, Marek Polacek wrote:
>> > +     if (gimple_code (stmt) == GIMPLE_TRY)
>> >         {
>> > +         /* A compiler-generated cleanup or a user-written try block.
>> > +            Try to get the first statement in its try-block, for better
>> > +            location.  */
>> > +         if ((seq = gimple_try_eval (stmt)))
>> > +           stmt = gimple_seq_first_stmt (seq);
>>
>> Should this loop?  If there are two variables declared, do we get two try
>> blocks?
>
> It looks like we get only one try block, so this doesn't have to loop.  But I
> at least added a new test to make sure we warn even with more decls.
>
> Bootstrapped/regtested on x86_64-linux, ok for trunk?

OK.

Jason

Reply via email to