[GHC] #9692: Template Haskell reification for Data Families is buggy?
GHC
ghc-devs at haskell.org
Thu Nov 8 15:38:22 UTC 2018
#9692: Template Haskell reification for Data Families is buggy?
-------------------------------------+-------------------------------------
Reporter: hpacheco | Owner: goldfire
Type: bug | Status: closed
Priority: normal | Milestone:
Component: Template Haskell | Version: 7.8.3
Resolution: fixed | Keywords:
| TemplateHaskell
Operating System: MacOS X | Architecture:
| Unknown/Multiple
Type of failure: None/Unknown | Test Case: th/T9692
Blocked By: | Blocking:
Related Tickets: | Differential Rev(s): Phab:D355
-------------------------------------+-------------------------------------
Comment (by Ryan Scott <ryan.gl.scott@…>):
In [changeset:"63a817074a8d49798bfd46a6545906fff143e924/ghc"
63a81707/ghc]:
{{{
#!CommitTicketReference repository="ghc"
revision="63a817074a8d49798bfd46a6545906fff143e924"
Fix #15845 by defining etaExpandFamInstLHS and using it
Summary:
Both #9692 and #14179 were caused by GHC being careless
about using eta-reduced data family instance axioms. Each of those
tickets were fixed by manually whipping up some code to eta-expand
the axioms. The same sort of issue has now caused #15845, so I
figured it was high time to factor out the code that each of these
fixes have in common.
This patch introduces the `etaExpandFamInstLHS` function, which takes
a family instance's type variables, LHS types, and RHS type, and
returns type variables and LHS types that have been eta-expanded if
necessary, in the case of a data family instance. (If it's a type
family instance, `etaExpandFamInstLHS` just returns the supplied type
variables and LHS types unchanged).
Along the way, I noticed that many references to
`Note [Eta reduction for data families]` (in `FamInstEnv`) had
slightly bitrotted (they either referred to a somewhat different
name, or claimed that the Note lived in a different module), so
I took the liberty of cleaning those up.
Test Plan: make test TEST="T9692 T15845"
Reviewers: goldfire, bgamari
Reviewed By: goldfire
Subscribers: rwbarton, carter
GHC Trac Issues: #15845
Differential Revision: https://phabricator.haskell.org/D5294
}}}
--
Ticket URL: <http://ghc.haskell.org/trac/ghc/ticket/9692#comment:8>
GHC <http://www.haskell.org/ghc/>
The Glasgow Haskell Compiler
More information about the ghc-tickets
mailing list