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.