This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Fix warnings in i386.c
- To: gcc-patches at gcc dot gnu dot org
- Subject: Fix warnings in i386.c
- From: Andreas Jaeger <aj at suse dot de>
- Date: Mon, 09 Jul 2001 09:41:15 +0200
I noticed these warnings:
/cvs/gcc/gcc/config/i386/i386.c: In function `print_operand':
/cvs/gcc/gcc/config/i386/i386.c:4237: warning: implicit declaration of function `final_forward_branch_p'
/cvs/gcc/gcc/config/i386/i386.c: In function `output_fix_trunc':
/cvs/gcc/gcc/config/i386/i386.c:4785: warning: unused variable `xops'
These warnings are fixed by the appended patch. Ok to commit if it
passes make bootstrap/check on i686-linux-gnu (just started
bootstrap)?
Andreas
2001-07-09 Andreas Jaeger <aj@suse.de>
* output.h: Add declaration of final_forward_branch_p.
* config/i386/i386.c (output_fix_trunc): Remove unused variable.
============================================================
Index: gcc/output.h
--- gcc/output.h 2001/07/08 19:55:56 1.62
+++ gcc/output.h 2001/07/09 07:40:57
@@ -1,7 +1,7 @@
/* Declarations for insn-output.c. These functions are defined in recog.c,
final.c, and varasm.c.
Copyright (C) 1987, 1991, 1994, 1997, 1998,
- 1999, 2000 Free Software Foundation, Inc.
+ 1999, 2000, 2001 Free Software Foundation, Inc.
This file is part of GNU CC.
@@ -115,6 +115,11 @@
/* Return nonzero if this function has no function calls. */
extern int leaf_function_p PARAMS ((void));
+
+/* Return 1 if branch is an forward branch.
+ Uses insn_shuid array, so it works only in the final pass. May be used by
+ output templates to customary add branch prediction hints. */
+extern int final_forward_branch_p PARAMS ((rtx));
/* Return 1 if this function uses only the registers that can be
safely renumbered. */
============================================================
Index: gcc/config/i386/i386.c
--- gcc/config/i386/i386.c 2001/07/09 06:10:02 1.280
+++ gcc/config/i386/i386.c 2001/07/09 07:40:58
@@ -4782,7 +4782,6 @@
{
int stack_top_dies = find_regno_note (insn, REG_DEAD, FIRST_STACK_REG) != 0;
int dimode_p = GET_MODE (operands[0]) == DImode;
- rtx xops[4];
/* Jump through a hoop or two for DImode, since the hardware has no
non-popping instruction. We used to do this a different way, but
--
Andreas Jaeger
SuSE Labs aj@suse.de
private aj@arthur.inka.de
http://www.suse.de/~aj