typecheck: Consider types containing coercions non-Typeable
authorBen Gamari <bgamari.foss@gmail.com>
Tue, 27 Jun 2017 14:29:33 +0000 (10:29 -0400)
committerBen Gamari <ben@smart-cactus.org>
Tue, 27 Jun 2017 16:55:06 +0000 (12:55 -0400)
This was previously a panic and caused #13871. I believe just saying
these types simply aren't Typeable should be correct.

Test Plan: Validate, check `T13871`

Reviewers: goldfire, austin

Subscribers: rwbarton, thomie, RyanGlScott

GHC Trac Issues: #13871

Differential Revision: https://phabricator.haskell.org/D3672

compiler/typecheck/TcTypeable.hs
testsuite/tests/typecheck/should_compile/all.T

index 64db97c..e7a427f 100644 (file)
@@ -439,7 +439,7 @@ typeIsTypeable (TyConApp tc args)   = tyConIsTypeable tc
 typeIsTypeable (ForAllTy{})         = False
 typeIsTypeable (LitTy _)            = True
 typeIsTypeable (CastTy{})           = False
-typeIsTypeable (CoercionTy{})       = panic "typeIsTypeable(Coercion)"
+typeIsTypeable (CoercionTy{})       = False
 
 -- | Maps kinds to 'KindRep' bindings. This binding may either be defined in
 -- some other module (in which case the @Maybe (LHsExpr Id@ will be 'Nothing')
index 5c3b323..72d33c1 100644 (file)
@@ -564,4 +564,4 @@ test('T13651', normal, compile, [''])
 test('T13785', normal, compile, [''])
 test('T13804', normal, compile, [''])
 test('T13822', normal, compile, [''])
-test('T13871', expect_broken(13871), compile, [''])
+test('T13871', normal, compile, [''])