On 04/04/2011 11:07 AM, Bernd Schmidt wrote:
I eventually managed to reproduce it, and even figured out what I'd stupidly been doing wrong with my bootstraps which caused me not to see this. There were a few places where last_scheduled_insn wasn't just examined on its own, but code wanted to walk backwards and forwards from it. This patch adapts them. I've also included Steven's patch from the bugzilla. Bootstrapped on i686-linux. Also tested, but I need to rerun those since there were other changes in the tree which were causing some failures. I've also built a powerpc-linux cross compiler, and compiled my set of examples with "-fsched-stalled-insns-dep=4 -msched-costly-dep=all -fsched-stalled-insns", with both patches included and with both patches removed, and found no code generation differences. I also did a run using "-fdbg-cnt=sched_insn:20", which exposed a preexisting bug in schedule_block (avoiding a requeue for the first insn, but not placing it back in the ready list). Ok after retest?
Ok, thanks.