On Thu, 2013-11-21 at 12:22 +0100, Oleg Endo wrote: > Sorry for the confusion. Initially I had it inserted right before > machine dependent reorg, at which point I was getting nullptr from > BLOCK_FOR_INSN all over the place. So not using it was the fix. > However, only after I've noticed that SH's machine dependent reorg > leaves the basic block structure in a 'broken' state (PR 59189) and thus > I moved the pass before sched2. > I'll try using BLOCK_FOR_INSN again.
... which won't work, since it sometimes also adds "unknown" ccreg_value entries where insn is NULL_RTX, but bb is valid. I'd like to keep it that way. It will be useful when moving sett / clrt insns around between basic blocks. I'll fix the confusing comment. Cheers, Oleg
