Re: [Patch PR78561 PowerPC] Revert to old behaviour for counting constant pools

2016-12-07 Thread Bernd Schmidt
On 12/07/2016 12:56 PM, Segher Boessenkool wrote: On Wed, Dec 07, 2016 at 10:31:14AM +, James Greenhalgh wrote: We always keep the entries in the constant pool after we first see them, and we figure out what gets used in the constant pool by sweeping through looking for referenced entries.

Re: [Patch PR78561 PowerPC] Revert to old behaviour for counting constant pools

2016-12-07 Thread Segher Boessenkool
Hi James, On Wed, Dec 07, 2016 at 10:31:14AM +, James Greenhalgh wrote: > We always keep the entries in the constant pool after we first see them, and > we figure out what gets used in the constant pool by sweeping through > looking for referenced entries. That means we can always answer the

[Patch PR78561 PowerPC] Revert to old behaviour for counting constant pools

2016-12-07 Thread James Greenhalgh
Hi, This patch is a bit of a shot in the dark, as I've been unable to reproduce Segher's failures reported in the PR. But, he has confirmed on the PR that this patch fixes the issue he was seeing. Before my patch set, rs6000 was reading the size of the constant pool at several points in