This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: GCC build failed for powerpc-eabisim with your patch on 2001-07-11T19:45:09Z.
- To: gcc-patches at gcc dot gnu dot org
- Subject: Re: GCC build failed for powerpc-eabisim with your patch on 2001-07-11T19:45:09Z.
- From: Jan Hubicka <jh at suse dot cz>
- Date: Thu, 12 Jul 2001 16:48:33 +0200
- References: <200107112134.f6BLYFm25700@maat.cygnus.com>
> cc1: warning: as it has already been specified as a system directory
> /maat/heart/tbox/cvs-gcc/gcc/gcc/libgcc2.c: In function `__addvsi3':
> /maat/heart/tbox/cvs-gcc/gcc/gcc/libgcc2.c:76: warning: implicit declaration of function `abort'
> /maat/heart/tbox/cvs-gcc/gcc/gcc/libgcc2.c:79: Internal compiler error in swap_condition, at jump.c:1989
> Please submit a full bug report,
> with preprocessed source if appropriate.
> See <URL:http://www.gnu.org/software/gcc/bugs.html> for instructions.
Hi,
I am installing this patch as obvious. It fixes the PPC failure I've introduced
by crossjumping work. When I was writing the code I made an thinko assuming
that swap_operands returns UNKNOWN on UNKNOWN so the following if will bail
out.
I apologize for the problem.
Honza
Thu Jul 12 16:46:10 CEST 2001 Jan Hubicka <jh@suse.cz>
* flow.c (outgoing_edges_match): Return early if condition reversal
failed.
Index: flow.c
===================================================================
RCS file: /cvs/gcc/egcs/gcc/flow.c,v
retrieving revision 1.419
diff -c -3 -p -r1.419 flow.c
*** flow.c 2001/07/11 19:42:33 1.419
--- flow.c 2001/07/12 14:45:29
*************** outgoing_edges_match (bb1, bb2)
*** 3393,3398 ****
--- 3371,3379 ----
code2 = reversed_comparison_code (cond2, bb2->end);
else
code2 = GET_CODE (cond2);
+
+ if (code2 == UNKNOWN)
+ return false;
/* See if we don have (cross) match in the codes and operands. */
match = ((code1 == code2