On Wed, Jun 20, 2018 at 4:37 PM Eric Botcazou <ebotca...@adacore.com> wrote: > > > There are fixes in this patch together with the new functionality - > > can you split > > those out? I'm thinking of the copy_edges_for_bb hunks as well as > > the expand_call_inline ones. > > Like this?
OK. Similar factoring of remap_location and copying/remapping edges goto_locus is a fix worth splitting out (and backporting eventually if a need arises). It interferes somewhat with the DECL_INGORED parts but IMHO is separate from those. Thanks, Richard. > > * tree-inline.c (copy_edges_for_bb): Minor tweak. > (maybe_move_debug_stmts_to_successors): Also reset the locus of the > debug statement when resetting its value. > (expand_call_inline): Copy the locus of the call onto the assignment > of > the return value, if any. Use local variable in more cases. > > -- > Eric Botcazou