On Apr 9, 2012, Alexandre Oliva <aol...@redhat.com> wrote: > The problem here is that a nondebug insn may be moved ahead of a useful > debug insn and clobber one of its inputs, rendering it useless, when > there's no good reason for the debug insn to be kept in place, other > than an accidental dependency on the previous insn when it happens to be > unrelated with the debug insn.
> Removing the extraneous dependency, that was thought to be a way to > reduce movement of debug insns, improves on this problem. It's not > clear that this artificial dependency really does any good, since odds > are that that previous insn may be pulled ahead anyway, in which case so > will debug insn (unless that would fail other of its deps, of course) > Retested. Ok? > for gcc/ChangeLog > from Alexandre Oliva <aol...@redhat.com> > * sched-deps.c (sched_analyze_insn): Don't force debug insns > to follow their original predecessors. Ping? http://gcc.gnu.org/ml/gcc-patches/2012-04/msg00418.html -- Alexandre Oliva, freedom fighter http://FSFLA.org/~lxoliva/ You must be the change you wish to see in the world. -- Gandhi Be Free! -- http://FSFLA.org/ FSF Latin America board member Free Software Evangelist Red Hat Brazil Compiler Engineer
