This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]

Fix warnings in i386.c



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


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]