Re: [PATCH] config/bfin/bfin.c (hwloop_optimize): Use return false instead of gcc_assert for checking jump_insn.

[Bernd Schmidt]

On 06/28/2015 04:15 PM, Chen Gang wrote:
For bfin looping optimization, after lsetup optimization, it can have
the correct lsetup related insns which causes gcc_assert for jump_insn.

I've been debugging this for a bit, and at least the explanation of the 
patch is wrong - it's finding an LSETUP for a different loop. There 
seems to be an inconsistency in the CFG, and it looks like it's caused 
by the unusual (?) situation that both arms out of a conditional branch 
lead directly to a hwloop candidate.

So, not OK until further investigation I think.


Bbernd