Re: [PATCH] Fix thunk expansion (PR ipa/64896)

[Yvan Roux <yvan dot roux at linaro dot org>]

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 ?


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.

Attachment: pr64896.diff
Description: Text document