Uros,

The fix looks good to me.

Best regards.
Yuri.

2013/5/3 Uros Bizjak <ubiz...@gmail.com>:
> Hello!
>
> Apparently, the wrong insn is used when adding insn dependencies. The
> "last" index gets updated to the found !NONDEBUG_INSN. Use "first_arg"
> that points to the right insn instead.
>
> 2013-05-03  Uros Bizjak  <ubiz...@gmail.com>
>
>     PR target/57106
>     * config/i386/i386.c (add_parameter_dependencies): Add dependence
>     between "first_arg" and "insn", not "last" and "insn".
>
> testsuite/ChangeLog:
>
> 2013-05-03  Uros Bizjak  <ubiz...@gmail.com>
>
>     * gcc.target/i386/pr57106.c: New test.
>
> Tested on x86_64-pc-linux-gnu {,-m32}. I will wait for Intel guys to
> check this patch with -fschedule-insns on their codes.
>
> Uros.

Reply via email to