#14880: GHC panic: updateRole -------------------------------------+------------------------------------- Reporter: RyanGlScott | Owner: goldfire Type: bug | Status: new Priority: normal | Milestone: 8.8.1 Component: Compiler (Type | Version: 8.2.2 checker) | Resolution: | Keywords: TypeInType Operating System: Unknown/Multiple | Architecture: Type of failure: Compile-time | Unknown/Multiple crash or panic | Test Case: Blocked By: | Blocking: Related Tickets: #15076 | Differential Rev(s): Phab:D4769 Wiki Page: | -------------------------------------+------------------------------------- Comment (by tdammers): Replying to [comment:112 goldfire]:
Where exactly is this change??
See changes to FV.hs and UniqDFM.hs here (which is the changeset for the original patch):
https://github.com/ghc/ghc/compare/bfc1fc2566944a455572303cbb2cbbf0c539c871....
Otherwise, go ahead with comment::110.
That's the entire changeset for T14880 though, isn't it? But we're really talking about fixing the insertion order for `FV` and `UniqDFM` for this step IIUC. So: Step 0: Fix FV / UniqDFM insertion order Step 1: Refactor `tyCoVars...` to use accumulator-style VarSet (Simon's patch) Step 2: Close over kinds at the end Step 3: "Everything else" -- Ticket URL: http://ghc.haskell.org/trac/ghc/ticket/14880#comment:114 GHC http://www.haskell.org/ghc/ The Glasgow Haskell Compiler