> 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

Reply via email to