https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81195
Dominique d'Humieres changed:
What|Removed |Added
CC||townsend at astro dot wisc.edu
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81195
Jim Wilson changed:
What|Removed |Added
Status|NEW |RESOLVED
Resolution|---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81195
--- Comment #10 from Jim Wilson ---
Author: wilson
Date: Mon Jun 26 21:44:50 2017
New Revision: 249668
URL: https://gcc.gnu.org/viewcvs?rev=249668=gcc=rev
Log:
Fix for SPEC CPu2017 621.wrf_s failure, add missing locking code.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81195
--- Comment #9 from Jim Wilson ---
Author: wilson
Date: Mon Jun 26 21:40:47 2017
New Revision: 249667
URL: https://gcc.gnu.org/viewcvs?rev=249667=gcc=rev
Log:
Fix for SPEC CPU2017 621.wrf_s failure, add missing locking code.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81195
Richard Biener changed:
What|Removed |Added
Keywords||openmp, wrong-code
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81195
--- Comment #8 from Thomas Koenig ---
(In reply to Jim Wilson from comment #7)
> I have reproduced the same problem from the gcc-7 branch.
I fully expect this bug to be in all active branches
of gfortran. We should commit a fix to all of them.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81195
Jim Wilson changed:
What|Removed |Added
Version|8.0 |7.1.1
--- Comment #7 from Jim Wilson ---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81195
--- Comment #6 from Jim Wilson ---
I should point out that there are currently four places that use
newunit_stack/newunit_tos without locking. Two of these places required
locking code to get wrf working.
The other two are init_units and
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81195
--- Comment #5 from Jerry DeLisle ---
(In reply to Thomas Koenig from comment #2)
> I was sort of waiting for the new SPEC suite to expose bugs :-)
>
> The patch looks obvious enough, even in the absence
> of a test case.
>
> Could you run a
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81195
--- Comment #4 from Jim Wilson ---
Suppose we move the locking inside the if statement. Suppose newunit_tos is 1.
Two threads hit the statement
if (newunit_tos)
at the same time, and both enter the if block. We then hit the lock. The
first
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81195
Dominique d'Humieres changed:
What|Removed |Added
Status|UNCONFIRMED |WAITING
Last reconfirmed|
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81195
Thomas Koenig changed:
What|Removed |Added
CC||jvdelisle at gcc dot gnu.org,
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81195
--- Comment #1 from Jim Wilson ---
Created attachment 41623
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=41623=edit
patch to make wrf_s work
13 matches
Mail list logo
