Re: [PATCH] pch: Add support for relocation of the PCH data [PR71934]

2021-12-09 Thread Eric Gallager via Gcc-patches
On Wed, Dec 8, 2021 at 6:10 PM Jeff Law via Gcc-patches wrote: > > > > On 12/7/2021 2:55 AM, Jakub Jelinek wrote: > > Hi! > > > > The following patch adds support for relocation of the PCH blob on PCH > > restore if we don't manage to get the preferred map slot for it. > > The GTY stuff knows

Re: [PATCH] pch: Add support for relocation of the PCH data [PR71934]

2021-12-09 Thread Jeff Law via Gcc-patches
On 12/9/2021 9:42 AM, Christophe Lyon via Gcc-patches wrote: Hi Jakub, On Thu, Dec 9, 2021 at 4:00 PM Jakub Jelinek via Gcc-patches < gcc-patches@gcc.gnu.org> wrote: On Wed, Dec 08, 2021 at 08:00:03AM +, Iain Sandoe wrote: On 7 Dec 2021, at 14:50, Jakub Jelinek via Gcc-patches <

Re: [PATCH] pch: Add support for relocation of the PCH data [PR71934]

2021-12-09 Thread Christophe Lyon via Gcc-patches
Hi Jakub, On Thu, Dec 9, 2021 at 4:00 PM Jakub Jelinek via Gcc-patches < gcc-patches@gcc.gnu.org> wrote: > On Wed, Dec 08, 2021 at 08:00:03AM +, Iain Sandoe wrote: > > > On 7 Dec 2021, at 14:50, Jakub Jelinek via Gcc-patches < > gcc-patches@gcc.gnu.org> wrote: > > The attached patch should

Re: [PATCH] pch: Add support for relocation of the PCH data [PR71934]

2021-12-09 Thread Iain Sandoe via Gcc-patches
> On 9 Dec 2021, at 14:59, Jakub Jelinek wrote: > > On Wed, Dec 08, 2021 at 08:00:03AM +, Iain Sandoe wrote: >>> On 7 Dec 2021, at 14:50, Jakub Jelinek via Gcc-patches >>> wrote: >> The attached patch should be applied before (or merged with) the change for >> relocation when it is

Re: [PATCH] pch: Add support for relocation of the PCH data [PR71934]

2021-12-09 Thread Jakub Jelinek via Gcc-patches
On Wed, Dec 08, 2021 at 08:00:03AM +, Iain Sandoe wrote: > > On 7 Dec 2021, at 14:50, Jakub Jelinek via Gcc-patches > > wrote: > The attached patch should be applied before (or merged with) the change for > relocation when it is applied - since the operation of the PCH hooks needs > some >

Re: [PATCH] pch: Add support for relocation of the PCH data [PR71934]

2021-12-08 Thread Jeff Law via Gcc-patches
On 12/8/2021 1:00 AM, Iain Sandoe wrote: On 7 Dec 2021, at 14:50, Jakub Jelinek via Gcc-patches wrote: On Tue, Dec 07, 2021 at 10:55:07AM +0100, Jakub Jelinek via Gcc-patches wrote: So, this patch instead builds a relocation table (sorted list of addresses in the blob which needs

Re: [PATCH] pch: Add support for relocation of the PCH data [PR71934]

2021-12-08 Thread Jeff Law via Gcc-patches
On 12/7/2021 2:55 AM, Jakub Jelinek wrote: Hi! The following patch adds support for relocation of the PCH blob on PCH restore if we don't manage to get the preferred map slot for it. The GTY stuff knows where all the pointers are, after all it relocates it once during PCH save from the

Re: [PATCH] pch: Add support for relocation of the PCH data [PR71934]

2021-12-08 Thread Iain Sandoe via Gcc-patches
> On 7 Dec 2021, at 14:50, Jakub Jelinek via Gcc-patches > wrote: > > On Tue, Dec 07, 2021 at 10:55:07AM +0100, Jakub Jelinek via Gcc-patches wrote: >> So, this patch instead builds a relocation table (sorted list of addresses >> in the blob which needs relocation) at PCH save time, stores it

Re: [PATCH] pch: Add support for relocation of the PCH data [PR71934]

2021-12-07 Thread Jakub Jelinek via Gcc-patches
On Tue, Dec 07, 2021 at 10:55:07AM +0100, Jakub Jelinek via Gcc-patches wrote: > So, this patch instead builds a relocation table (sorted list of addresses > in the blob which needs relocation) at PCH save time, stores it in a very > compact form into the gch file and upon restore, adjusts

[PATCH] pch: Add support for relocation of the PCH data [PR71934]

2021-12-07 Thread Jakub Jelinek via Gcc-patches
Hi! The following patch adds support for relocation of the PCH blob on PCH restore if we don't manage to get the preferred map slot for it. The GTY stuff knows where all the pointers are, after all it relocates it once during PCH save from the addresses where it was initially allocated to