Re: [PATCH] i386: More use reference of struct ix86_frame to avoid copy

2018-01-16 Thread H.J. Lu
On Tue, Jan 16, 2018 at 8:09 AM, H.J. Lu wrote: > On Tue, Jan 16, 2018 at 7:03 AM, Martin Liška wrote: >> On 01/16/2018 01:35 PM, H.J. Lu wrote: >>> On Tue, Jan 16, 2018 at 3:40 AM, H.J. Lu wrote: This patch has been used with my Spectre backport for GCC 7 for many weeks and has been c

Re: [PATCH] i386: More use reference of struct ix86_frame to avoid copy

2018-01-16 Thread H.J. Lu
On Tue, Jan 16, 2018 at 7:03 AM, Martin Liška wrote: > On 01/16/2018 01:35 PM, H.J. Lu wrote: >> On Tue, Jan 16, 2018 at 3:40 AM, H.J. Lu wrote: >>> This patch has been used with my Spectre backport for GCC 7 for many >>> weeks and has been checked into GCC 7 branch. Should I revert it on >>> GC

Re: [PATCH] i386: More use reference of struct ix86_frame to avoid copy

2018-01-16 Thread Martin Liška
On 01/16/2018 01:35 PM, H.J. Lu wrote: > On Tue, Jan 16, 2018 at 3:40 AM, H.J. Lu wrote: >> This patch has been used with my Spectre backport for GCC 7 for many >> weeks and has been checked into GCC 7 branch. Should I revert it on >> GCC 7 branch or check it into trunk? > > Ada build failed wit

Re: [PATCH] i386: More use reference of struct ix86_frame to avoid copy

2018-01-16 Thread H.J. Lu
On Tue, Jan 16, 2018 at 3:40 AM, H.J. Lu wrote: > This patch has been used with my Spectre backport for GCC 7 for many > weeks and has been checked into GCC 7 branch. Should I revert it on > GCC 7 branch or check it into trunk? Ada build failed with this on trunk: raised STORAGE_ERROR : stack o

[PATCH] i386: More use reference of struct ix86_frame to avoid copy

2018-01-16 Thread H.J. Lu
This patch has been used with my Spectre backport for GCC 7 for many weeks and has been checked into GCC 7 branch. Should I revert it on GCC 7 branch or check it into trunk? H.J. --- When there is no need to make a copy of ix86_frame, we can use reference of struct ix86_frame to avoid copy.