On 03/05/2012 08:54 AM, Aldy Hernandez wrote:
>    region_worklist =
>      (struct tm_region **) xcalloc (sizeof (struct tm_region *),
> -                               n_basic_blocks + NUM_FIXED_BLOCKS + 2);
> +                               last_basic_block + NUM_FIXED_BLOCKS);

This is ok.

I was confused for a moment by the "worklist" variable name, which
suggests a queue.  I'd also suggest that you change to use a vec,
rather than callocing yourself, and would have caught the memory
error earlier.


Reply via email to