[PATCH] Don't cast XNEWVEC's result

Marek Polacek polacek@redhat.com
Thu Sep 20 10:05:00 GMT 2012


While reading tree-ssa-threadupdate.c I noticed that we're casting
XNEWVEC's result.  This isn't done anywhere else and the macro
does the cast internally anyway, thus there's no need to repeat that.
Also there was a typo in copyright years, so I fixed that as well
while at it.  Regtested/bootstrapped on x86_64.

2012-09-19  Marek Polacek  <polacek@redhat.com>

	* tree-ssa-threadupdate.c (mark_threaded_blocks): Don't cast XNEWVEC's
	result.
	(create_edge_and_update_destination_phis): Likewise.

--- gcc/tree-ssa-threadupdate.c.mp	2012-09-19 20:12:49.711262969 +0200
+++ gcc/tree-ssa-threadupdate.c	2012-09-19 20:17:42.427287843 +0200
@@ -1,5 +1,5 @@
 /* Thread edges through blocks and update the control flow and SSA graphs.
-   Copyright (C) 2004, 2005, 2006, 2007, 2008, 2010, 201
+   Copyright (C) 2004, 2005, 2006, 2007, 2008, 2010, 2011, 2012
    Free Software Foundation, Inc.
 
 This file is part of GCC.
@@ -362,7 +362,7 @@ create_edge_and_update_destination_phis
 
   if (rd->outgoing_edge->aux)
     {
-      e->aux = (edge *) XNEWVEC (edge, 2);
+      e->aux = XNEWVEC (edge, 2);
       THREAD_TARGET(e) = THREAD_TARGET (rd->outgoing_edge);
       THREAD_TARGET2(e) = THREAD_TARGET2 (rd->outgoing_edge);
     }
@@ -1143,7 +1143,7 @@ mark_threaded_blocks (bitmap threaded_bl
   for (i = 0; i < VEC_length (edge, threaded_edges); i += 3)
     {
       edge e = VEC_index (edge, threaded_edges, i);
-      edge *x = (edge *) XNEWVEC (edge, 2);
+      edge *x = XNEWVEC (edge, 2);
 
       e->aux = x;
       THREAD_TARGET (e) = VEC_index (edge, threaded_edges, i + 1);

	Marek



More information about the Gcc-patches mailing list