Re: [PATCH 10/10] Compress short ranges into source_location

2015-11-04 Thread Dodji Seketeli
[...] > diff --git a/libcpp/line-map.c b/libcpp/line-map.c [...] > + > + /* Any ordinary locations ought to be "pure" at this point: no > + compressed ranges. */ > + linemap_assert (locus < RESERVED_LOCATION_COUNT > + || locus >= LINE_MAP_MAX_LOCATION_WITH_COLS > +

Re: [PATCH 10/10] Compress short ranges into source_location

2015-10-29 Thread Jeff Law
On 10/23/2015 02:41 PM, David Malcolm wrote: gcc/ada/ChangeLog: * gcc-interface/trans.c (Sloc_to_locus): Add line_table param when calling linemap_position_for_line_and_column. gcc/ChangeLog: * input.c (dump_line_table_statistics): Dump stats on how many ranges w

[PATCH 10/10] Compress short ranges into source_location

2015-10-23 Thread David Malcolm
The attached patch implements a bit-packing scheme so that short ranges can be stored directly within a 32-bit source_location (aka location_t) without needing to use the ad-hoc table. The intent is to mitigate the overhead introduced in the earlier patch that added ranges for all tokens in libcpp