On Sun, Nov 24, 2019 at 11:39:05AM -0600, Segher Boessenkool wrote: > On Sun, Nov 24, 2019 at 03:19:49PM +0100, Bernd Schmidt wrote: > > - /* This continue is deliberate. We do not want the uses of the > > - jump put into reg_next_use because it is not considered safe to > > - combine a preincrement with a jump. */ > > - if (JUMP_P (insn)) > > - continue; > > Huh, I wonder where that came from. More archaelogy :-)
It came in as r115135 on the dataflow-branch. Patch is at https://gcc.gnu.org/ml/gcc-patches/2006-07/msg00037.html . And that is where I lost track. But. Allowing autoinc into jump insns means those jump insns may then eventually need an output reload; it may just have been because of that? Segher
