Let's do #9374 before implementing this patch. Right now SAT is only run when you say `-fstatic-argument-transformation` (ie almost never) so it would be hard to test this (doubtless excellent) refactoring
#9561: Clean up mergeSATInfo -------------------------------------+------------------------------------- Reporter: dfeuer | Owner: Type: task | Status: patch Priority: normal | Milestone: 7.8.4 Component: Compiler | Version: 7.9 Resolution: | Keywords: Operating System: | Architecture: Unknown/Multiple Unknown/Multiple | Difficulty: Easy (less than 1 Type of failure: | hour) None/Unknown | Blocked By: Test Case: | Related Tickets: Blocking: | Differential Revisions: | -------------------------------------+------------------------------------- Comment (by dfeuer): Replying to [comment:2 simonpj]: thoroughly.
Simon
I understand your concern, but if you look at it, this is a ''very'' limited change. I think two pairs of eyeballs and the type checker passing it should be pretty convincing. -- Ticket URL: http://ghc.haskell.org/trac/ghc/ticket/9561#comment:3 GHC http://www.haskell.org/ghc/ The Glasgow Haskell Compiler