[commit: ghc] master: Disallow class instances for synonyms (c750808)

[git at git.haskell.org]

Repository : ssh://git@git.haskell.org/ghc

On branch  : master
Link       : http://ghc.haskell.org/trac/ghc/changeset/c7508083388a71d76a5b6f1e46adfbcffba74b96/ghc

>---------------------------------------------------------------

commit c7508083388a71d76a5b6f1e46adfbcffba74b96
Author: Simon Peyton Jones <simonpj at microsoft.com>
Date:   Tue Feb 21 15:00:03 2017 +0000

    Disallow class instances for synonyms
    
    See Trac #13267 and Note [Instances and constraint synonyms]
    in TcValidity.
    
    We can't easily do a perfect job, because the rename is really trying
    to do its lookup too early.  But this is at least an improvement.


>---------------------------------------------------------------

c7508083388a71d76a5b6f1e46adfbcffba74b96
 compiler/typecheck/TcValidity.hs        | 45 ++++++++++++++++++++++++++++-----
 testsuite/tests/polykinds/T13267.hs     | 10 ++++++++
 testsuite/tests/polykinds/T13267.stderr | 10 ++++++++
 testsuite/tests/polykinds/all.T         |  1 +
 4 files changed, 60 insertions(+), 6 deletions(-)

diff --git a/compiler/typecheck/TcValidity.hs b/compiler/typecheck/TcValidity.hs
index f7cb319..6f9c3fa 100644
--- a/compiler/typecheck/TcValidity.hs
+++ b/compiler/typecheck/TcValidity.hs
@@ -1236,11 +1236,42 @@ validDerivPred tv_set pred
 ************************************************************************
 -}
 
+{- Note [Instances and constraint synonyms]
+~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
+Currently, we don't allow instances for constraint synonyms at all.
+Consider these (Trac #13267):
+  type C1 a = Show (a -> Bool)
+  instance C1 Int where    -- I1
+    show _ = "ur"
+
+This elicits "show is not a (visible) method of class C1", which isn't
+a great message. But it comes from the renamer, so it's hard to improve.
+
+This needs a bit more care:
+  type C2 a = (Show a, Show Int)
+  instance C2 Int           -- I2
+
+If we use (splitTyConApp_maybe tau) in checkValidInstance to decompose
+the instance head, we'll expand the synonym on fly, and it'll look like
+  instance (%,%) (Show Int, Show Int)
+and we /really/ don't want that.  So we carefully do /not/ expand
+synonyms, by matching on TyConApp directly.
+-}
+
 checkValidInstance :: UserTypeCtxt -> LHsSigType Name -> Type
                    -> TcM ([TyVar], ThetaType, Class, [Type])
 checkValidInstance ctxt hs_type ty
-  | Just (clas,inst_tys) <- getClassPredTys_maybe tau
-  , inst_tys `lengthIs` classArity clas
+  | not is_tc_app
+  = failWithTc (text "Instance head is not headed by a class")
+
+  | isNothing mb_cls
+  = failWithTc (vcat [ text "Illegal instance for a" <+> text (tyConFlavour tc)
+                     , text "A class instance must be for a class" ])
+
+  | not arity_ok
+  = failWithTc (text "Arity mis-match in instance head")
+
+  | otherwise
   = do  { setSrcSpan head_loc (checkValidInstHead ctxt clas inst_tys)
         ; traceTc "checkValidInstance {" (ppr ty)
         ; checkValidTheta ctxt theta
@@ -1269,11 +1300,13 @@ checkValidInstance ctxt hs_type ty
         ; traceTc "End checkValidInstance }" empty
 
         ; return (tvs, theta, clas, inst_tys) }
-
-  | otherwise
-  = failWithTc (text "Malformed instance head:" <+> ppr tau)
   where
-    (tvs, theta, tau) = tcSplitSigmaTy ty
+    (tvs, theta, tau)    = tcSplitSigmaTy ty
+    is_tc_app            = case tau of { TyConApp {} -> True; _ -> False }
+    TyConApp tc inst_tys = tau   -- See Note [Instances and constraint synonyms]
+    mb_cls               = tyConClass_maybe tc
+    Just clas            = mb_cls
+    arity_ok             = inst_tys `lengthIs` classArity clas
 
         -- The location of the "head" of the instance
     head_loc = getLoc (getLHsInstDeclHead hs_type)
diff --git a/testsuite/tests/polykinds/T13267.hs b/testsuite/tests/polykinds/T13267.hs
new file mode 100644
index 0000000..cfc7efb
--- /dev/null
+++ b/testsuite/tests/polykinds/T13267.hs
@@ -0,0 +1,10 @@
+{-# LANGUAGE ConstraintKinds, FlexibleContexts, FlexibleInstances, MultiParamTypeClasses #-}
+module T13267 where
+
+type C1 a = (Show (a -> Bool))
+
+instance C1 Int where
+
+type C2 a = (Show Bool, Show Int)
+
+instance C2 Int where
diff --git a/testsuite/tests/polykinds/T13267.stderr b/testsuite/tests/polykinds/T13267.stderr
new file mode 100644
index 0000000..ff6d7fd
--- /dev/null
+++ b/testsuite/tests/polykinds/T13267.stderr
@@ -0,0 +1,10 @@
+
+T13267.hs:6:10: error:
+    • Illegal instance for a type synonym
+      A class instance must be for a class
+    • In the instance declaration for ‘C1 Int’
+
+T13267.hs:10:10: error:
+    • Illegal instance for a type synonym
+      A class instance must be for a class
+    • In the instance declaration for ‘C2 Int’
diff --git a/testsuite/tests/polykinds/all.T b/testsuite/tests/polykinds/all.T
index 4cdcc17..270aea3 100644
--- a/testsuite/tests/polykinds/all.T
+++ b/testsuite/tests/polykinds/all.T
@@ -154,3 +154,4 @@ test('T12668', normal, compile, [''])
 test('T12718', normal, compile, [''])
 test('T12444', normal, compile_fail, [''])
 test('T12885', normal, compile, [''])
+test('T13267', normal, compile_fail, [''])