This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [-fcompare-debug] find jump before debug insns in expand
- From: Alexandre Oliva <aoliva at redhat dot com>
- To: gcc-patches at gcc dot gnu dot org
- Cc: rdsandiford at googlemail dot com
- Date: Tue, 03 Jan 2017 18:00:56 -0200
- Subject: Re: [-fcompare-debug] find jump before debug insns in expand
- Authentication-results: sourceware.org; auth=none
- References: <orzij83e6b.fsf@lxoliva.fsfla.org> <8737h0w80p.fsf@googlemail.com>
On Jan 3, 2017, Richard Sandiford <rdsandiford@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