On Jan  3, 2017, Richard Sandiford <rdsandif...@googlemail.com> wrote:

>>        && (last = get_last_insn ())
>> -      && JUMP_P (last))
>> +      && (JUMP_P (last)
>> +      || (DEBUG_INSN_P (last)
>> +          && JUMP_P (prev_nondebug_insn (last)))))

> Would it be worth adding a get_last_nondebug_insn in case other patterns
> like this crop up?

I didn't think so.  Most of the RTL passes use the BB-based interfaces
nowadays, so it seemed that cfgexpand would be pretty much the only
place where this could be used.  That was my reasoning anyway; I didn't
actually check that this was indeed the case.

-- 
Alexandre Oliva, freedom fighter    http://FSFLA.org/~lxoliva/
You must be the change you wish to see in the world. -- Gandhi
Be Free! -- http://FSFLA.org/   FSF Latin America board member
Free Software Evangelist|Red Hat Brasil GNU Toolchain Engineer

Reply via email to