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 <pola...@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
