public inbox for gcc-patches@gcc.gnu.org
 help / color / mirror / Atom feed
* PATCH: Fix bootstrap
@ 2007-07-31 18:47 H.J. Lu
  2007-07-31 19:44 ` Revital1 Eres
  0 siblings, 1 reply; 3+ messages in thread
From: H.J. Lu @ 2007-07-31 18:47 UTC (permalink / raw)
  To: gcc-patches; +Cc: eres

Bootstrap was broken:

cc1: warnings being treated as
errors/net/gnu-13/export/gnu/src/gcc/gcc/gcc/ddg.c: In function
‘add_cross_iteration_register_deps’:
/net/gnu-13/export/gnu/src/gcc/gcc/gcc/ddg.c:242: error: unused
variable ‘bb_info’
make[5]: *** [ddg.o] Error 1
make[5]: *** Waiting for unfinished jobs....

when configured with "--enable-checking=assert" due to

http://gcc.gnu.org/ml/gcc-patches/2007-07/msg02136.html

I am checking in this patch as an obvious fix.

H.J.
----
2007-07-31  H.J. Lu  <hongjiu.lu@intel.com>

	* ddg.c (add_cross_iteration_register_deps): Declare bb_info
	only if ENABLE_CHECKING is defined.

--- gcc/ddg.c.foo	2007-07-31 11:33:33.000000000 -0700
+++ gcc/ddg.c	2007-07-31 11:31:36.000000000 -0700
@@ -239,7 +239,9 @@ add_cross_iteration_register_deps (ddg_p
   rtx def_insn = DF_REF_INSN (last_def);
   ddg_node_ptr last_def_node = get_node_of_insn (g, def_insn);
   ddg_node_ptr use_node;
+#ifdef ENABLE_CHECKING
   struct df_rd_bb_info *bb_info = DF_RD_BB_INFO (g->bb);
+#endif
   struct df_ref *first_def = df_bb_regno_first_def_find (g->bb, regno);
 
   gcc_assert (last_def_node);

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: PATCH: Fix bootstrap
  2007-07-31 18:47 PATCH: Fix bootstrap H.J. Lu
@ 2007-07-31 19:44 ` Revital1 Eres
  0 siblings, 0 replies; 3+ messages in thread
From: Revital1 Eres @ 2007-07-31 19:44 UTC (permalink / raw)
  To: H.J. Lu; +Cc: gcc-patches


Sorry about that.

Thanks for the fix,
Revital

gcc-patches-owner@gcc.gnu.org wrote on 31/07/2007 21:40:45:

> Bootstrap was broken:
>
> cc1: warnings being treated as
> errors/net/gnu-13/export/gnu/src/gcc/gcc/gcc/ddg.c: In function
> ‘add_cross_iteration_register_deps’:
> /net/gnu-13/export/gnu/src/gcc/gcc/gcc/ddg.c:242: error: unused
> variable ‘bb_info’
> make[5]: *** [ddg.o] Error 1
> make[5]: *** Waiting for unfinished jobs....
>
> when configured with "--enable-checking=assert" due to
>
> http://gcc.gnu.org/ml/gcc-patches/2007-07/msg02136.html
>
> I am checking in this patch as an obvious fix.
>
> H.J.
> ----
> 2007-07-31  H.J. Lu  <hongjiu.lu@intel.com>
>
>    * ddg.c (add_cross_iteration_register_deps): Declare bb_info
>    only if ENABLE_CHECKING is defined.
>
> --- gcc/ddg.c.foo   2007-07-31 11:33:33.000000000 -0700
> +++ gcc/ddg.c   2007-07-31 11:31:36.000000000 -0700
> @@ -239,7 +239,9 @@ add_cross_iteration_register_deps (ddg_p
>    rtx def_insn = DF_REF_INSN (last_def);
>    ddg_node_ptr last_def_node = get_node_of_insn (g, def_insn);
>    ddg_node_ptr use_node;
> +#ifdef ENABLE_CHECKING
>    struct df_rd_bb_info *bb_info = DF_RD_BB_INFO (g->bb);
> +#endif
>    struct df_ref *first_def = df_bb_regno_first_def_find (g->bb, regno);
>
>    gcc_assert (last_def_node);

^ permalink raw reply	[flat|nested] 3+ messages in thread

* PATCH: Fix bootstrap
@ 2009-05-11 13:51 H.J. Lu
  0 siblings, 0 replies; 3+ messages in thread
From: H.J. Lu @ 2009-05-11 13:51 UTC (permalink / raw)
  To: gcc-patches; +Cc: irar

Hi,

On Linux/ia32, I got

/export/gnu/import/svn/gcc-test/bld/./prev-gcc/xgcc
-B/export/gnu/import/svn/gcc-test/bld/./prev-gcc/
-B/usr/local/i686-pc-linux-gnu/bin/ -c  -g -O2 -fomit-frame-pointer
-DIN_GCC   -W -Wall -Wwrite-strings -Wstrict-prototypes
-Wmissing-prototypes -Wcast-qual -Wold-style-definition -Wc++-compat
-Wmissing-format-attribute -pedantic -Wno-long-long -Wno-variadic-macros
-Wno-overlength-strings -Werror -fno-common  -DHAVE_CONFIG_H -I. -I.
-I../../src-trunk/gcc -I../../src-trunk/gcc/.
-I../../src-trunk/gcc/../include -I../../src-trunk/gcc/../libcpp/include
-I../../src-trunk/gcc/../libdecnumber
-I../../src-trunk/gcc/../libdecnumber/bid -I../libdecnumber
../../src-trunk/gcc/tree-vect-stmts.c -o tree-vect-stmts.o
cc1: warnings being treated as errors
../../src-trunk/gcc/tree-vect-data-refs.c: In function
'vect_analyze_group_access':
../../src-trunk/gcc/tree-vect-data-refs.c:1513: error: comparison
between signed and unsigned integer expressions
make[6]: *** [tree-vect-data-refs.o] Error 1

I am checking in this patch to fix it.

Thanks.


H.J.
---
Index: ChangeLog
===================================================================
--- ChangeLog	(revision 147374)
+++ ChangeLog	(working copy)
@@ -1,3 +1,8 @@
+2009-05-11  H.J. Lu  <hongjiu.lu@intel.com>
+
+	* tree-vect-data-refs.c (vect_analyze_group_access): Use
+	HOST_WIDE_INT for gap.
+
 2009-05-11  Ira Rosen  <irar@il.ibm.com>
 
 	PR tree-optimization/40074
Index: tree-vect-data-refs.c
===================================================================
--- tree-vect-data-refs.c	(revision 147374)
+++ tree-vect-data-refs.c	(working copy)
@@ -1424,11 +1424,11 @@ vect_analyze_group_access (struct data_r
       /* First stmt in the interleaving chain. Check the chain.  */
       gimple next = DR_GROUP_NEXT_DR (vinfo_for_stmt (stmt));
       struct data_reference *data_ref = dr;
-      unsigned int count = 1, gaps = 0;
+      unsigned int count = 1;
       tree next_step;
       tree prev_init = DR_INIT (data_ref);
       gimple prev = stmt;
-      HOST_WIDE_INT diff, count_in_bytes;
+      HOST_WIDE_INT diff, count_in_bytes, gaps = 0;
 
       while (next)
         {

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2009-05-11 13:51 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2007-07-31 18:47 PATCH: Fix bootstrap H.J. Lu
2007-07-31 19:44 ` Revital1 Eres
2009-05-11 13:51 H.J. Lu

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).