This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH] Fix debug info for expr and jump stmt
On Wed, Oct 31, 2012 at 11:00:26AM +0100, Richard Biener wrote:
> 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.
Yeah. But please also check gdb testsuite for this kind of patches.
Jakub