[GHC] #15168: GHC HEAD crashes on Windows 64 after the SRT overhaul (May 16, 2018 and newer)
GHC
ghc-devs at haskell.org
Wed May 23 16:05:49 UTC 2018
#15168: GHC HEAD crashes on Windows 64 after the SRT overhaul (May 16, 2018 and
newer)
-------------------------------------+-------------------------------------
Reporter: awson | Owner: (none)
Type: bug | Status: new
Priority: highest | Milestone: 8.6.1
Component: Compiler | Version: 8.5
Resolution: | Keywords:
Operating System: Windows | Architecture: x86_64
Type of failure: Compile-time | (amd64)
crash or panic | Test Case:
Blocked By: | Blocking:
Related Tickets: | Differential Rev(s): Phab:D4721
Wiki Page: |
-------------------------------------+-------------------------------------
Comment (by Simon Marlow <marlowsd@…>):
In [changeset:"d424d4a46a729f8530e9273282d22b6b8f34daaa/ghc"
d424d4a4/ghc]:
{{{
#!CommitTicketReference repository="ghc"
revision="d424d4a46a729f8530e9273282d22b6b8f34daaa"
Fix a bug in SRT generation
Summary:
I had good intentions, but they were not being followed. In particular,
this comment:
```
--- - we never resolve a reference to a CAF to the contents of its SRT,
since
--- the point of SRTs is to keep CAFs alive.
```
was not true, because we updated the srtMap after generating the SRT
for a CAF. Therefore it was possible for another CAF to refer to an
earlier CAF, and the reference to the earlier CAF would be shortcutted
to refer to its SRT instead of pointing to the CAF itself.
The fix is just to not update the srtMap when generating the SRT for a
CAF, but I also refactored the code and comments around this to be a bit
better organised.
Test Plan: Harbourmaster
Reviewers: bgamari, michalt, simonpj, erikd
Subscribers: rwbarton, thomie, carter
GHC Trac Issues: #15173, #15168
Differential Revision: https://phabricator.haskell.org/D4721
}}}
--
Ticket URL: <http://ghc.haskell.org/trac/ghc/ticket/15168#comment:14>
GHC <http://www.haskell.org/ghc/>
The Glasgow Haskell Compiler
More information about the ghc-tickets
mailing list