> Alternatively, perhaps all we care about is either i3, or NEXT_INSN of the > last debug_insn propagate_for_debug changed in an interesting way. Thus > propagate_for_debug could return the last DEBUG_INSN it changed, and caller > decide either that no updating of last_modified_debug_insn is needed (that > is the case of the pair of propagate_for_debug calls from first to last > which just make it a lowpart subreg), or it would update it.
Maybe. I was more or less waiting for Alexandre's take on all this. :-) > 2011-04-04 Jakub Jelinek <ja...@redhat.com> > > PR debug/48343 > * combine.c (combine_instructions): Add last_combined_insn, > update it if insn is after it, pass it to all try_combine > calls. > (try_combine): Add last_combined_insn parameter, pass it instead of > i3 to propagate_for_debug. > > * gcc.dg/torture/pr48343.c: New test. This version is fine by me at least, thanks. -- Eric Botcazou