[PR debug/46576] don't reset prev_nonjump_insn at debug insns

Richard Guenther richard.guenther@gmail.com
Wed Dec 8 10:56:00 GMT 2010


On Wed, Dec 8, 2010 at 5:11 AM, Alexandre Oliva <aoliva@redhat.com> wrote:
> mark_all_labels() retains information from the prior insn to attempt to
> fill in the JUMP_LABEL of a computed jump.  If there happens to be a
> debug insn between them, we won't fill in the JUMP_LABEL, and
> -fcompare-debug fails.  Presumably in larger testcases this may also
> lead to codegen differences.
>
> This patch skips debug insns while scanning basic blocks, so that we
> don't reset prev_nonjump_insn and don't otherwise waste time at them.
>
> Regstrapped on x86_64-linux-gnu.  Ok to install?

Ok.

Thanks,
Richard.

>
>
> --
> 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
>
>



More information about the Gcc-patches mailing list