This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH] Fix thunk expansion (PR ipa/64896)
- From: Yvan Roux <yvan dot roux at linaro dot org>
- To: Jan Hubicka <hubicka at ucw dot cz>
- Cc: Jakub Jelinek <jakub at redhat dot com>, Richard Biener <rguenther at suse dot de>, "gcc-patches at gcc dot gnu dot org" <gcc-patches at gcc dot gnu dot org>
- Date: Tue, 10 Mar 2015 18:35:40 +0100
- Subject: Re: [PATCH] Fix thunk expansion (PR ipa/64896)
- Authentication-results: sourceware.org; auth=none
- References: <20150206202331 dot GO1746 at tucnak dot redhat dot com> <20150206203709 dot GA22441 at kam dot mff dot cuni dot cz> <CAD57uCfDKomop66KRd9RGhwH6aX=mKnJqjpVm_LHfyoTbsDCkw at mail dot gmail dot com>
Hi
On 9 March 2015 at 17:07, Yvan Roux <yvan.roux@linaro.org> wrote:
> Hi,
>
> As added in the PR, this issue is also present on 4.9 branch and
> affects at least arm-linux-gnueabihf target (as reported in PR61207).
>
> I've backported it in the 4.9 branch with the attached patch. The
> difference with the trunk code is due the code introduced by PR63587
> fix (I didn't checked on power7, on which the PR was initially
> reported, but I didn't managed to reproduce the issue for arm targets
> on 4.9 branch).
>
> Boostrapped on x86_64, and tested on arm/aarch64 targets (regression
> testing is ongoing). is ok for 4.9 branch when validation is done ?
So bootstrapped/regtested on x86_64 and cross-compiled/regtested on
aarch64-linux-gnu
arm-linux-gnueabihf
armeb-linux-gnueabihf
i686-linux-gnu
> Thanks
> Yvan
>
> gcc/
> 2015-03-09 Yvan Roux <yvan.roux@linaro.org>
>
> Backport from trunk r220489.
> 2015-02-06 Jakub Jelinek <jakub@redhat.com>
>
> PR ipa/64896
> * cgraphunit.c (cgraph_node::expand_thunk): If
> restype is not is_gimple_reg_type nor the thunk_fndecl
> returns aggregate_value_p, set restmp to a temporary variable
> instead of resdecl.
>
> gcc/testsuite/
> 2015-03-09 Yvan Roux <yvan.roux@linaro.org>
>
> Backport from trunk r220489.
> 2015-02-06 Jakub Jelinek <jakub@redhat.com>
>
> PR ipa/64896
> * g++.dg/ipa/pr64896.C: New test.