Re: [PATCH v2] Re: PR62304 (was Re: (Still) ICE for cris-elf at r214710)

2014-08-30 Thread David Malcolm
On Fri, 2014-08-29 at 23:41 -0600, Jeff Law wrote: On 08/29/14 12:07, David Malcolm wrote: Yes: I made various mistakes in reorg.c and resource.c where I assumed that a JUMP_LABEL(insn) was an insn, whereas the existing code is set up to handle RETURN nodes. Well, it would seem to me

[PATCH v2] Re: PR62304 (was Re: (Still) ICE for cris-elf at r214710)

2014-08-29 Thread David Malcolm
On Fri, 2014-08-29 at 18:15 +0200, Hans-Peter Nilsson wrote: From: David Malcolm dmalc...@redhat.com Date: Fri, 29 Aug 2014 17:33:54 +0200 On Fri, 2014-08-29 at 16:48 +0200, Hans-Peter Nilsson wrote: Sorry, but that didn't help. I still get the exact same error. (Yep, I

Re: [PATCH v2] Re: PR62304 (was Re: (Still) ICE for cris-elf at r214710)

2014-08-29 Thread Hans-Peter Nilsson
From: David Malcolm dmalc...@redhat.com Date: Fri, 29 Aug 2014 20:07:04 +0200 BTW, in another email in the thread you said: Thanks for the heads-up. BTW, the ChangeLog entries should say what not why; that goes into a comment in the source. OK. Where possible I've added comments in

Re: [PATCH v2] Re: PR62304 (was Re: (Still) ICE for cris-elf at r214710)

2014-08-29 Thread Jeff Law
On 08/29/14 12:07, David Malcolm wrote: Yes: I made various mistakes in reorg.c and resource.c where I assumed that a JUMP_LABEL(insn) was an insn, whereas the existing code is set up to handle RETURN nodes. Well, it would seem to me that reorg is being totally braindead in mixing and