This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [patch] tree-ssa-live.c: Use a heap-allocated array instead ofVARRAY (Part 2)
- From: Nathan Sidwell <nathan at codesourcery dot com>
- To: Kazu Hirata <kazu at cs dot umass dot edu>
- Cc: gcc-patches at gcc dot gnu dot org, amacleod at redhat dot com
- Date: Sat, 23 Apr 2005 17:20:18 +0100
- Subject: Re: [patch] tree-ssa-live.c: Use a heap-allocated array instead ofVARRAY (Part 2)
- Organization: CodeSourcery LLC
- References: <20050423.105459.21309938.kazu@cs.umass.edu>
Kazu Hirata wrote:
Hi,
Attached is a patch to allocate partition_link and tpa_nodes on heap.
We do not change the sizes of these two arrays. They don't "escape"
build_tree_conflict_graph, either. So we can simply allocate these
two arrays on heap without using the updated VEC API.
Y'know, all things being equal, why not use the VEC API here? you'll
get virtually the same code (using _VEC_quick_foo or whatever), and with
checking enabled you're guaranteed not to go wandering off the end.
just my 2pennies.
nathan
--
Nathan Sidwell :: http://www.codesourcery.com :: CodeSourcery LLC
nathan@codesourcery.com :: http://www.planetfall.pwp.blueyonder.co.uk