https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84155
--- Comment #23 from Jürgen Reuter ---
All our code is fine also with this fix.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84155
--- Comment #22 from Jürgen Reuter ---
I just started building r257550 of the trunk and will check our code. I'll
report back of there are any issues.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84155
Paul Thomas changed:
What|Removed |Added
Status|REOPENED|RESOLVED
Resolution|---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84155
--- Comment #20 from Paul Thomas ---
Author: pault
Date: Sat Feb 10 18:16:14 2018
New Revision: 257550
URL: https://gcc.gnu.org/viewcvs?rev=257550&root=gcc&view=rev
Log:
2018-02-10 Paul Thomas
PR fortran/84141
PR fortran/8415
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84155
--- Comment #19 from Paul Thomas ---
fferent types but who knows.
>
> I suggest to remove the caching from gfc_get_dtype.
Indeed, it is the caching that is the source of the problem. I reverted the fix
and removed the caching from gfc_get_dtype
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84155
Richard Biener changed:
What|Removed |Added
Status|RESOLVED|REOPENED
Resolution|FIXED
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84155
Paul Thomas changed:
What|Removed |Added
Status|NEW |RESOLVED
Resolution|---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84155
--- Comment #16 from Thomas Koenig ---
The statements are removed upon conversion to gimple:
--- Falsch/a.f90.004t.gimple2018-02-03 15:08:29.370147886 +0100
+++ Korrekt/a.f90.004t.gimple 2018-02-03 15:07:16.428178637 +0100
@@ -893,6 +893,9
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84155
--- Comment #15 from Paul Thomas ---
Author: pault
Date: Sat Feb 3 14:06:44 2018
New Revision: 257356
URL: https://gcc.gnu.org/viewcvs?rev=257356&root=gcc&view=rev
Log:
2018-02-03 Paul Thomas
PR fortran/84141
PR fortran/8415
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84155
--- Comment #14 from paul.richard.thomas at gmail dot com ---
Hi Dominique,
Thanks for doing that. It was to have been my final step in the process.
I will commit the patch and then will go back to diagnose why an
unchanged tree dump yields dif
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84155
--- Comment #13 from Dominique d'Humieres ---
> There is an important caveat to this fix, which has me very worried:
> On top of removal of uncalled code making the bug disappear, I cannot
> see any difference in the tree dump between the working
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84155
--- Comment #12 from Thomas Koenig ---
(In reply to Jürgen Reuter from comment #11)
> There was another test case that I submitted for #84141. It still failed
> after the first preliminary fix.
This one also works with the patch from comment #8
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84155
--- Comment #11 from Jürgen Reuter ---
There was another test case that I submitted for #84141. It still failed after
the first preliminary fix.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84155
--- Comment #10 from Paul Thomas ---
Created attachment 43328
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=43328&action=edit
Patch for the bug
This patch fixes the PR and PR84141. The dejagnuified version of the reproducer
will appear wi
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84155
--- Comment #9 from Jürgen Reuter ---
This fixes almost all of our unit and functional test, but not all of them.
There are still 19 functional tests failing, all of them seem to have to do
with some sort of I/O . And one unit tests, which I cann
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84155
--- Comment #8 from Paul Thomas ---
A temporary fix:
Index: ../trunk/gcc/fortran/trans-io.c
===
*** ../trunk/gcc/fortran/trans-io.c (revision 257261)
--- ../trunk/gcc/fortran/tr
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84155
Paul Thomas changed:
What|Removed |Added
Assignee|unassigned at gcc dot gnu.org |pault at gcc dot gnu.org
--- Commen
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84155
Dominique d'Humieres changed:
What|Removed |Added
Keywords||wrong-code
Priority|P3
18 matches
Mail list logo