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 se

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 matching

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

2014-08-29 Thread Hans-Peter Nilsson
> From: David Malcolm > 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 the n

[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 > > 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 double-checked that

Re: PR62304 (was Re: (Still) ICE for cris-elf at r214710)

2014-08-29 Thread Hans-Peter Nilsson
> From: David Malcolm > 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 double-checked that I didn't goof testing...) Famous last words... > Fully identical, or j

PR62304 (was Re: (Still) ICE for cris-elf at r214710)

2014-08-29 Thread David Malcolm
On Fri, 2014-08-29 at 16:48 +0200, Hans-Peter Nilsson wrote: > > From: David Malcolm > > Date: Fri, 29 Aug 2014 13:26:59 +0200 > > On Fri, 2014-08-29 at 06:13 +0200, Hans-Peter Nilsson wrote: > > > /tmp/hpautotest-gcc1/cris-elf/gccobj/./gcc/xgcc > > > -B/tmp/hpautotest-gcc1/cris-elf/gccobj/./gcc/

Re: (Still) ICE for cris-elf at r214710

2014-08-29 Thread Hans-Peter Nilsson
> From: David Malcolm > Date: Fri, 29 Aug 2014 13:26:59 +0200 > On Fri, 2014-08-29 at 06:13 +0200, Hans-Peter Nilsson wrote: > > /tmp/hpautotest-gcc1/cris-elf/gccobj/./gcc/xgcc > > -B/tmp/hpautotest-gcc1/cris-elf/gccobj/./gcc/ -nostdinc > > -B/tmp/hpautotest-gcc1/cris-elf/gccobj/cris-elf/newlib/

Re: (Still) ICE for cris-elf at r214710

2014-08-29 Thread Hans-Peter Nilsson
> From: David Malcolm > Date: Fri, 29 Aug 2014 13:40:49 +0200 > > Patch attached, which fixes the above testcase; bootstrap in progress: > > > > gcc/ > > * resource.h (mark_target_live_regs): Undo erroneous conversion > > of second param of r214693, converting it back from rtx_insn * to

Re: (Still) ICE for cris-elf at r214710

2014-08-29 Thread David Malcolm
On Fri, 2014-08-29 at 07:26 -0400, David Malcolm wrote: > On Fri, 2014-08-29 at 06:13 +0200, Hans-Peter Nilsson wrote: > > Sorry for the context-less mail but I didn't find a proper > > obvious gcc-patches-message to reply to. (Also, I can't log > > into bugzilla because to enter a PR as there app

Re: (Still) ICE for cris-elf at r214710

2014-08-29 Thread David Malcolm
On Fri, 2014-08-29 at 06:13 +0200, Hans-Peter Nilsson wrote: > Sorry for the context-less mail but I didn't find a proper > obvious gcc-patches-message to reply to. (Also, I can't log > into bugzilla because to enter a PR as there appears to have > been some SSL changes such that my old firefox an

(Still) ICE for cris-elf at r214710

2014-08-28 Thread Hans-Peter Nilsson
Sorry for the context-less mail but I didn't find a proper obvious gcc-patches-message to reply to. (Also, I can't log into bugzilla because to enter a PR as there appears to have been some SSL changes such that my old firefox and gcc.gnu.org can no longer agree on a cipher or something.) But, si