On Thu, Aug 18, 2016 at 04:01:42PM +0200, Jakub Jelinek wrote:
> On Thu, Aug 18, 2016 at 03:50:07PM +0200, Marek Polacek wrote:
> > +    case GIMPLE_BIND:
> > +      {
> > +   gbind *bind = as_a <gbind *> (stmt);
> > +   return last_stmt_in_scope (
> > +            gimple_seq_last_stmt (gimple_bind_body (bind)));
> > +      }
> > +
> > +    case GIMPLE_TRY:
> > +      {
> > +   gtry *try_stmt = as_a <gtry *> (stmt);
> > +   return last_stmt_in_scope (
> > +            gimple_seq_last_stmt (gimple_try_eval (try_stmt)));
> 
> Just a minor formatting detail.
>       stmt = gimple_seq_last_stmt (gimple_try_eval (try_stmt));
>       return last_stmt_in_scope (stmt);
> and similarly above might be nicer.  Or do the tail recursion by hand?
> No need to repost for that.

Fixed as suggested, thanks.  Yes -- trailing ( are super ugly.

        Marek

Reply via email to