[PATCH] Combine location with block using block_locations

Diego Novillo dnovillo@google.com
Fri Sep 7 11:08:00 GMT 2012


On 2012-09-06 05:46 , Richard Guenther wrote:
> On Wed, Aug 22, 2012 at 1:54 AM, Dehao Chen <dehao@google.com> wrote:
>> On Tue, Aug 21, 2012 at 6:25 AM, Richard Guenther
>> <richard.guenther@gmail.com> wrote:
>>> On Mon, Aug 20, 2012 at 3:18 AM, Dehao Chen <dehao@google.com> wrote:
>>>> ping....
>>>
>>> Conceptually I like the change.  Can a libcpp maintainer please have a 2nd
>>> look?
>>>
>>> Dehao, did you do any compile-time and memory-usage benchmarks?
>>
>> I don't have a memory benchmarks at hand. But I've tested it through
>> some huge apps, each of which takes more than 1 hour to build on a
>> modern machine. None of them had observed noticeable memory footprint
>> and compile time increase.
>
> Thanks.
>
> I'd like to see a 2nd "ok", but hereby I give a first one ;)  It's a big change
> but definitely a good one.

Dehao, could you attach a final, and complete, patch?  I've seen 3 or 4 
patches in this thread.  The latest version is only a fragment of the 
full patch.


Thanks.  Diego.



More information about the Gcc-patches mailing list