This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: patch to fix PR65805
- From: Jakub Jelinek <jakub at redhat dot com>
- To: Vladimir Makarov <vmakarov at redhat dot com>
- Cc: "gcc-patches at gcc dot gnu dot org" <gcc-patches at gcc dot gnu dot org>
- Date: Mon, 20 Apr 2015 10:05:31 +0200
- Subject: Re: patch to fix PR65805
- Authentication-results: sourceware.org; auth=none
- References: <55343F20 dot 9080009 at redhat dot com>
- Reply-to: Jakub Jelinek <jakub at redhat dot com>
On Sun, Apr 19, 2015 at 07:49:52PM -0400, Vladimir Makarov wrote:
> The followingpatch fixes
>
> https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65805
>
> The problem occurred when SP was changed between the original insn and
> rematerialized one and the rematerialized insn contained a reg which will be
> substituted by SP. In this case difference between sp offset and the
> previous sp offset was used twice.
>
> The patch was bootstrapped and tested on x86/x86-64 and ppc64.
>
> It is hard to create a test for the PR which will be stable. So the test is
> absent.
>
> Committed to the trunk as rev. 222223.
>
> Jakub, the problem is present in GCC5 RC too. Should I commit it into
> gcc-5-branch too?
Yes, but only after the 5.1 release (Wednesday), I think it is too risky
this late before the release.
Jakub