* Richard Biener, 2012-11-28 :
> You need to stream the id with LTO, no? Also increasing the size
> of line_map_ordinary might not be the most welcom change ...
Thanks for your feedback Richard, I'll take care of adding the missing
LTO pieces, and improve documentation of the flag.
As for the size impact on line_map_ordinary, indeed that's an additional
int here, but I'm not sure we allocate so many objects of that kind that
the increase is significant.
Now, an alternative might be to have an array of instance IDs stored at
the struct line_maps level, with the same indices at the set of ordinary
maps, which would be allocated only when flag_debug_instances is used;
when it is not used the cost would then be a single NULL pointer in
struct line_maps. Would that be acceptable?
Side note: I first tried to implement this option using a vec<int>,
but it appears that vec.h cannot currently be used from within libcpp.
So, I'll have to either change that (but that seems like a pretty
significant change) or else manually manage this array through xrealloc.
Thomas.
--
Thomas Quinot, Ph.D. ** qui...@adacore.com ** Senior Software Engineer
AdaCore -- Paris, France -- New York, USA
