Ping.
On 11 March 2015 at 16:38, Yvan Roux <yvan.r...@linaro.org> wrote: > Hi, > > >>>> PR ipa/65236 >>>> * cgraphunit.c (cgraph_node::expand_thunk): Enable return slot >>>> opt. >> >> This bugfix adds ipa-icf-6.C test which failed on 4.9 branch as >> ipa-icf is not backported on that branch. Is the bugfix still >> relevant and we can dropped the testcase ? >> >>>> PR ipa/64813 >>>> * cgraphunit.c (cgraph_node::expand_thunk): Do not create >>>> a return value for call to a function that is noreturn. >>>> >>>> PR ipa/63595 >>>> * cgraphunit.c (cgraph_node::expand_thunk): DECL_BY_REFERENCE >>>> is correctly handled for thunks created by IPA ICF. >>>> >>>> PR ipa/63587 >>>> * cgraphunit.c (cgraph_node::expand_thunk): Only VAR_DECLs are put >>>> to local declarations. >>>> * function.c (add_local_decl): Implementation moved from header >>>> file, assert introduced for tree type. >>>> * function.h: Likewise. > > Here is the two patches that backport PR ipa/63587 and PR ipa/64813 > fixes in 4.9 branch. The 2 others introduce test cases that check > ipa-icf pass dumps, so I'm not sure if the code has to be backported. > > bootstrapped/regtested on x86_64 and cross-compiled/regtested on > aarch64-linux-gnu > arm-linux-gnueabihf > armeb-linux-gnueabihf > i686-linux-gnu > > Ok for 4.9 ? > > Thanks > Yvan > > ----- PR 63587 ----- > gcc/ > 2015-03-11 Yvan Roux <yvan.r...@linaro.org> > > Backport from trunk r216841. > 2014-10-29 Martin Liska <mli...@suse.cz> > > PR ipa/63587 > * cgraphunit.c (cgraph_node::expand_thunk): Only VAR_DECLs are put > to local declarations. > * function.c (add_local_decl): Implementation moved from header > file, assert introduced for tree type. > * function.h: Likewise. > > gcc/testsuite/ > 2015-03-11 Yvan Roux <yvan.r...@linaro.org> > > Backport from trunk r216841. > 2014-10-29 Martin Liska <mli...@suse.cz> > > PR ipa/63587 > * g++.dg/ipa/pr63587-1.C: New test. > * g++.dg/ipa/pr63587-2.C: New test. > > ----- PR 64813 ----- > 2015-03-11 Yvan Roux <yvan.r...@linaro.org> > > Backport from trunk r220616. > 2015-02-11 Martin Liska <mli...@suse.cz> > > PR ipa/64813 > * cgraphunit.c (cgraph_node::expand_thunk): Do not create > a return value for call to a function that is noreturn.