Re: [PATCH 1/2] fix PR49847 ICE-on-HAVE_cc0 regression from PR50780 changes

2013-10-29 Thread Richard Biener
On Wed, Aug 28, 2013 at 10:06 PM, Jeff Law wrote: > On 08/28/2013 03:50 AM, Mikael Pettersson wrote: >> >> This patch fixes an ICE that occurs in #ifdef HAVE_cc0 code. The ICE >> breaks both Java and Ada bootstrap on m68k-linux. There is also a >> tiny C++ test case in the BZ entry. >> >> The IC

Re: [PATCH 1/2] fix PR49847 ICE-on-HAVE_cc0 regression from PR50780 changes

2013-08-29 Thread Mikael Pettersson
Jeff Law writes: > >* cse.c (fold_rtx) : If prev_insn_cc0 is NULL > >don't call equiv_constant on it. > I can't help but feel something different needs to be done here. It's > always been the case that those two insns are expected to be contiguous > and there's going to be code all

Re: [PATCH 1/2] fix PR49847 ICE-on-HAVE_cc0 regression from PR50780 changes

2013-08-28 Thread Jeff Law
On 08/28/2013 03:50 AM, Mikael Pettersson wrote: This patch fixes an ICE that occurs in #ifdef HAVE_cc0 code. The ICE breaks both Java and Ada bootstrap on m68k-linux. There is also a tiny C++ test case in the BZ entry. The ICE is triggered by the PR middle-end/50780 changes in r180192. As exp