> This patch fixes debug info for expr and jump stmt. It would be nice to have testcases...
> * cfgexpand.c (set_expr_location_r): New callback function. > (gimple_assign_rhs_to_tree): Walk the expr recursively. > (expand_call_stmt): Likewise. > (expand_gimple_stmt_1): Likewise. This cannot be right. You're going to propagate a toplevel location above locations at deeper level that can be different and should prevail. It would be far better to set the right location at node creation instead of patching it up during RTL expansion. -- Eric Botcazou