diff mbox

[bootstrap-O1] add initializers to avoid warnings at -O1

Message ID orvatw3e5r.fsf@lxoliva.fsfla.org
State New
Headers show

Commit Message

Alexandre Oliva Jan. 3, 2017, 5:28 a.m. UTC 
Building with the bootstrap-O1 configuration option fails to compile a
number of files due to AFAICT false-positive warnings about uses of
uninitialized variables.

This patch adds dummy initializers to silence them all.

Regstrapped on x86_64-linux-gnu and i686-linux-gnu.  OK to install?

for  gcc/ChangeLog

	* multiple_target.c (create_dispatcher_calls): Init e_next.
	* tree-ssa-loop-split.c (split_loop): Init border.
	* tree-vect-loop.c (vect_determine_vectorization_factor): Init
	scalar_type.
---
 gcc/multiple_target.c     |    2 +-
 gcc/tree-ssa-loop-split.c |    2 +-
 gcc/tree-vect-loop.c      |    2 +-
 3 files changed, 3 insertions(+), 3 deletions(-)

Comments

Jeff Law Jan. 3, 2017, 5:40 p.m. UTC | #1 
On 01/02/2017 10:28 PM, Alexandre Oliva wrote:
> Building with the bootstrap-O1 configuration option fails to compile a
> number of files due to AFAICT false-positive warnings about uses of
> uninitialized variables.
>
> This patch adds dummy initializers to silence them all.
>
> Regstrapped on x86_64-linux-gnu and i686-linux-gnu.  OK to install?
>
> for  gcc/ChangeLog
>
> 	* multiple_target.c (create_dispatcher_calls): Init e_next.
> 	* tree-ssa-loop-split.c (split_loop): Init border.
> 	* tree-vect-loop.c (vect_determine_vectorization_factor): Init
> 	scalar_type.
Most likely these are due to either not running VRP (and thus the jump 
threading within VRP) or a throttled jump threading elsewhere.

OK.

jeff
diff mbox

Patch

diff --git a/gcc/multiple_target.c b/gcc/multiple_target.c
index 4ead029..bb118b5 100644
--- a/gcc/multiple_target.c
+++ b/gcc/multiple_target.c
@@ -42,7 +42,7 @@  static void
 create_dispatcher_calls (struct cgraph_node *node)
 {
   cgraph_edge *e;
-  cgraph_edge *e_next;
+  cgraph_edge *e_next = NULL;
 
   /* We need to remember NEXT_CALLER as it could be modified in the loop.  */
   for (e = node->callers; e ;e = (e == NULL) ? e_next : e->next_caller)
diff --git a/gcc/tree-ssa-loop-split.c b/gcc/tree-ssa-loop-split.c
index 84c0627..cb2808f 100644
--- a/gcc/tree-ssa-loop-split.c
+++ b/gcc/tree-ssa-loop-split.c
@@ -497,7 +497,7 @@  split_loop (struct loop *loop1, struct tree_niter_desc *niter)
   unsigned i;
   bool changed = false;
   tree guard_iv;
-  tree border;
+  tree border = NULL_TREE;
   affine_iv iv;
 
   bbs = get_loop_body (loop1);
diff --git a/gcc/tree-vect-loop.c b/gcc/tree-vect-loop.c
index 6e8b89c..ced9d96 100644
--- a/gcc/tree-vect-loop.c
+++ b/gcc/tree-vect-loop.c
@@ -183,7 +183,7 @@  vect_determine_vectorization_factor (loop_vec_info loop_vinfo)
   basic_block *bbs = LOOP_VINFO_BBS (loop_vinfo);
   unsigned nbbs = loop->num_nodes;
   unsigned int vectorization_factor = 0;
-  tree scalar_type;
+  tree scalar_type = NULL_TREE;
   gphi *phi;
   tree vectype;
   unsigned int nunits;