https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63851
--- Comment #14 from Martin Liška marxin at gcc dot gnu.org ---
Author: marxin
Date: Tue Dec 23 09:30:20 2014
New Revision: 219042
URL: https://gcc.gnu.org/viewcvs?rev=219042root=gccview=rev
Log:
Fix for PR ipa/63851 and ipa/63852.
PR
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63851
Martin Liška marxin at gcc dot gnu.org changed:
What|Removed |Added
Status|NEW |RESOLVED
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63851
--- Comment #11 from Martin Liška marxin at gcc dot gnu.org ---
Hello.
The problem is caused by missing static-chain in newly created GIMPLE_CALL in
expand_thunk:
fpi (struct array7_integer(kind=4) xx1, struct array7_t yy1)
{
bb 2:
fpa
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63851
--- Comment #12 from howarth at bromo dot med.uc.edu ---
I can confirm that the proposed patch from comment 11 along with the one from
comment 5 eliminates all of the fortran testsuite regressions on
x86_64-apple-darwin14 at r218871.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63851
--- Comment #13 from Dominique d'Humieres dominiq at lps dot ens.fr ---
I can confirm that the proposed patch from comment 11 along with the one
from comment 5 eliminates all of the fortran test suite regressions
on x86_64-apple-darwin14 at
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63851
--- Comment #8 from Dominique d'Humieres dominiq at lps dot ens.fr ---
Results with the patch in comment 5 at
https://gcc.gnu.org/ml/gcc-testresults/2014-12/msg02164.html.
Note that the test gfortran.dg/assumed_rank_10.f90 fails at run time when
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63851
--- Comment #9 from howarth at bromo dot med.uc.edu ---
(In reply to Dominique d'Humieres from comment #8)
Results with the patch in comment 5 at
https://gcc.gnu.org/ml/gcc-testresults/2014-12/msg02164.html.
Note that the test
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63851
--- Comment #10 from howarth at bromo dot med.uc.edu ---
(In reply to Dominique d'Humieres from comment #8)
Results with the patch in comment 5 at
https://gcc.gnu.org/ml/gcc-testresults/2014-12/msg02164.html.
Note that the test
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63851
--- Comment #5 from Martin Liška marxin at gcc dot gnu.org ---
Yes, IPA ICF should respect 'restrict' attribute.
May I ask you to rerun test suite with applied:
diff --git a/gcc/ipa-icf-gimple.c b/gcc/ipa-icf-gimple.c
index ec0290a..98f38ee
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63851
--- Comment #6 from Dominique d'Humieres dominiq at lps dot ens.fr ---
Yes, IPA ICF should respect 'restrict' attribute.
May I ask you to rerun test suite with applied:
My machine is busy regtesting 4.8.4, but a quick test shows that your
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63851
howarth at bromo dot med.uc.edu changed:
What|Removed |Added
CC||howarth at bromo dot
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63851
Tobias Burnus burnus at gcc dot gnu.org changed:
What|Removed |Added
CC||burnus at gcc
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63851
--- Comment #4 from Francois-Xavier Coudert fxcoudert at gcc dot gnu.org ---
(In reply to Martin Liška from comment #2)
There's a pair of functions 'g' and 'h' that are proved by IPA ICF to be
equal and thunk is created (darwin does not have
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63851
--- Comment #2 from Martin Liška marxin at gcc dot gnu.org ---
OK, I can reproduce the problem on x86_64 with following simple patch applied:
diff --git a/gcc/ipa-icf.c b/gcc/ipa-icf.c
index 4875dec..c985052 100644
--- a/gcc/ipa-icf.c
+++
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63851
Dominique d'Humieres dominiq at lps dot ens.fr changed:
What|Removed |Added
Target|x86_64-apple-darwin14
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63851
Richard Biener rguenth at gcc dot gnu.org changed:
What|Removed |Added
Keywords||wrong-code
16 matches
Mail list logo
