Re: [PATCH] Fix debug info for expr and jump stmt

[Richard Biener <richard dot guenther at gmail dot com>]

On Tue, Oct 30, 2012 at 6:27 PM, Dehao Chen <dehao@google.com> wrote:
>> gcc/ChangeLog:
>> 2012-10-25  Dehao Chen  <dehao@google.com>
>>
>>         * tree-eh.c (do_return_redirection): Set location for jump statement.
>>         (do_goto_redirection): Likewise.
>>         (frob_into_branch_around): Likewise.
>>         (lower_try_finally_nofallthru): Likewise.
>>         (lower_try_finally_copy): Likewise.
>>         (lower_try_finally_switch): Likewise.
>>         * expr.c (store_expr): Use current insn location instead of expr
>>         location.
>>         (expand_expr_real): Likewise.
>>         (expand_expr_real_1): Likewise.
>>
>> gcc/testsuite/ChangeLog:
>> 2012-10-25  Dehao Chen  <dehao@google.com>
>>
>>         * g++.dg/debug/dwarf2/block.C: New testcase.
>
> This patch bootstrapped and passed gcc regression tests.
>
> Okay for trunk?

It looks sensible to me.  If nobody objects within 24h the patch is ok.

Thanks,
Richard.

> Thanks,
> Dehao