Re: [PR 78140] Reuse same IPA bits and VR info

2017-02-28 Thread Richard Biener
On Tue, Feb 28, 2017 at 3:08 PM, Martin Jambor wrote: > Hi, > > On Mon, Feb 27, 2017 at 10:34:38AM +0100, Richard Biener wrote: >> On Wed, Feb 22, 2017 at 11:11 AM, Martin Jambor wrote: >> > Hello, >> > >> > this is a fix for PR 78140 which is about LTO WPA of Firefox taking >> > 1GB memory more

Re: [PR 78140] Reuse same IPA bits and VR info

2017-02-28 Thread Martin Jambor
Hi, On Mon, Feb 27, 2017 at 10:34:38AM +0100, Richard Biener wrote: > On Wed, Feb 22, 2017 at 11:11 AM, Martin Jambor wrote: > > Hello, > > > > this is a fix for PR 78140 which is about LTO WPA of Firefox taking > > 1GB memory more than gcc 6. > > > > It works by reusing the ipa_bits and value_ra

Re: [PR 78140] Reuse same IPA bits and VR info

2017-02-28 Thread jh
According to my measurements, the patch saves about 1.2 GB of memory. The problem is that some change last week (between revision 245382 and 245595) has more than invalidated this: | compiler| WPA mem (GB) | |-+--| | gcc 6 branch|

Re: [PR 78140] Reuse same IPA bits and VR info

2017-02-27 Thread Richard Biener
On Wed, Feb 22, 2017 at 11:11 AM, Martin Jambor wrote: > Hello, > > this is a fix for PR 78140 which is about LTO WPA of Firefox taking > 1GB memory more than gcc 6. > > It works by reusing the ipa_bits and value_range that we previously > had directly in jump functions and which are just too big

Re: [PR 78140] Reuse same IPA bits and VR info

2017-02-23 Thread Martin Jambor
Hi, On Wed, Feb 22, 2017 at 11:11:14AM +0100, Martin Jambor wrote: > Hello, > > this is a fix for PR 78140 which is about LTO WPA of Firefox taking > 1GB memory more than gcc 6. > > It works by reusing the ipa_bits and value_range that we previously > had directly in jump functions and which are

[PR 78140] Reuse same IPA bits and VR info

2017-02-22 Thread Martin Jambor
Hello, this is a fix for PR 78140 which is about LTO WPA of Firefox taking 1GB memory more than gcc 6. It works by reusing the ipa_bits and value_range that we previously had directly in jump functions and which are just too big to be allocated for each actual argument in all of Firefox. Reusing