This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: Fix PR debug/33921, lost lexical blocks


On Sat, Oct 27, 2007 at 01:48:29PM -0400, Daniel Jacobowitz wrote:
> The code in reemit_insn_block_notes chooses the innermost block to
> assign to a SEQUENCE based on the locators of all the insns in the
> sequence.  It completely ignores the locator of the sequence itself.
> So I think the easiest fix is to leave the locator on both the
> SEQUENCE and the leading JUMP_INSN.
> 
> I can't readily bootstrap on any platform with delay slots.  Dave,
> would you rather I test this with a MIPS cross compiler or do it
> yourself on hppa-linux?

> 2007-10-27  Daniel Jacobowitz  <dan@codesourcery.com>
> 
> 	* reorg.c (emit_delay_sequence): Do not clear the locator on
> 	the jump instruction.

This is a very simple patch, and partial reversion of an earlier
patch; so I've committed it as obvious after the hppa-linux testing
succeeded.  Sorry for the breakage.

-- 
Daniel Jacobowitz
CodeSourcery


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]