This is the mail archive of the
mailing list for the GCC project.
Re: [PATCH] var-tracking.c speedups
On Tue, Jun 23, 2009 at 03:30:52PM +0200, Richard Guenther wrote:
> The rtl.c parts are ok if you add a warning comment to both
> rtx_equal_p and rtx_equal_cb_p that they should be kept in sync.
Ok, will do.
> Is it worth having the vui static buffer _and_ an allocated variant?
> Why not just unconditionally use the allocated variant?
The cost is
static struct variable_union_info vui_buf;
if (src_l + dst_l <= (int) ARRAY_SIZE (vui_buf))
vui = vui_buf;
i.e. 6 lines, for something that will avoid the malloc for 100% of
cases during gcc bootstrap and at least 99.9% for other projects as well
seems worth to me, but if you think 6 lines are too much for that...