search for: r236629

...lData). As a result, during object emission we save ~6 pointers per MCSymbol, eliminate lookup indirection between MCSymbol and MCSymbolData, and avoid allocation overhead for MCSymbolData. I've measured ~4% memory savings on `llc` with this patch (using the same -flto -g input as r236642 and r236629 before it), dropping memory usage from 1058 MB down to 1017 MB. Before I clean this up and commit it (obviously in more incremental patches (and I'll do the same cleanup for MCSection)), I wanted to confirm the direction. -------------- next part -------------- A non-text attachment was scrub...

...829 MB - merge DIEAbbrevData into DIEValue - 0004: 829 MB - refactor - 0005: 829 MB - refactor - 0006: 829 MB - refactor - 0007: 764 MB - change DIE::Values to a linked list - 0008: 756 MB - change DIE::Children to a linked list (Still measuring memory on `llc` for `-flto -g`; details in r236629.) @Eric, you mentioned offline you'd like to have a look at this proposal before I proceed -- obviously I've been impatient ;). Let me know if I'm okay to move forward and start committing (modulo a couple of these that I'll want a review from David and Fred on). -------------- n...

With just those four patches, memory usage went *up* slightly. Add in the 5th patch (which does #2 below), and we get an overall memory drop of 4%. The intermediate result of a memory increase makes sense. While the first four patches reduce the number of (and size of) `DIEValue` allocations, they increase the cost of the `SmallVector` overhead. 0005 (attached) squeezes the abbreviation data

On Fri, Apr 15, 2016 at 4:04 PM, Duncan P. N. Exon Smith < dexonsmith at apple.com> wrote: > > > On 2016-Apr-15, at 14:53, David Blaikie <dblaikie at gmail.com> wrote: > > > > > > > >> On Fri, Apr 15, 2016 at 2:27 PM, Duncan P. N. Exon Smith < > dexonsmith at apple.com> wrote: > >> > >> > On 2016-Apr-15, at 10:27, David