Add bang patterns
[ghc-hetmet.git] / ghc / compiler / hsSyn / HsBinds.lhs
index 7437f09..69b75b4 100644 (file)
 %
 \section[HsBinds]{Abstract syntax: top-level bindings and signatures}
 
-Datatype for: @HsBinds@, @Bind@, @Sig@, @MonoBinds@.
+Datatype for: @BindGroup@, @Bind@, @Sig@, @Bind@.
 
 \begin{code}
 module HsBinds where
 
 #include "HsVersions.h"
 
-import {-# SOURCE #-} HsExpr ( HsExpr, pprExpr,
-                              Match,  pprFunBind,
-                              GRHSs,  pprPatBind )
+import {-# SOURCE #-} HsExpr ( HsExpr, pprExpr, LHsExpr,
+                              MatchGroup, pprFunBind,
+                              GRHSs, pprPatBind )
+import {-# SOURCE #-} HsPat  ( LPat )
 
--- friends:
-import HsImpExp                ( pprHsVar )
-import HsPat           ( Pat )
-import HsTypes         ( HsType )
-import PprCore         ( {- instance Outputable (Expr a) -} )
-
---others:
+import HsTypes         ( LHsType, PostTcType )
+import Type            ( Type )
 import Name            ( Name )
-import PrelNames       ( isUnboundName )
-import NameSet         ( NameSet, elemNameSet, nameSetToList )
-import BasicTypes      ( RecFlag(..), FixitySig(..), Activation(..), IPName )
+import NameSet         ( NameSet, elemNameSet )
+import BasicTypes      ( IPName, RecFlag(..), InlineSpec(..), Fixity )
 import Outputable      
-import SrcLoc          ( SrcLoc )
-import Var             ( TyVar )
-import Class            ( DefMeth (..) )
+import SrcLoc          ( Located(..), SrcSpan, unLoc )
+import Util            ( sortLe )
+import Var             ( TyVar, DictId, Id )
+import Bag             ( Bag, emptyBag, isEmptyBag, bagToList, unionBags, unionManyBags )
 \end{code}
 
 %************************************************************************
 %*                                                                     *
-\subsection{Bindings: @HsBinds@}
+\subsection{Bindings: @BindGroup@}
 %*                                                                     *
 %************************************************************************
 
-The following syntax may produce new syntax which is not part of the input,
-and which is instead a translation of the input to the typechecker.
-Syntax translations are marked TRANSLATION in comments. New empty
-productions are useful in development but may not appear in the final
-grammar.
-
-Collections of bindings, created by dependency analysis and translation:
+Global bindings (where clauses)
 
 \begin{code}
-data HsBinds id                -- binders and bindees
-  = EmptyBinds
-  | ThenBinds  (HsBinds id) (HsBinds id)
-
-  | MonoBind                   -- A mutually recursive group
-       (MonoBinds id)
-       [Sig id]                -- Empty on typechecker output, Type Signatures
-       RecFlag
-
-  | IPBinds                    -- Implcit parameters
-                               -- Not allowed at top level
-       [(IPName id, HsExpr id)]
-       Bool            -- True <=> this was a 'with' binding
-                       --  (tmp, until 'with' is removed)
-\end{code}
+data HsLocalBinds id   -- Bindings in a 'let' expression
+                       -- or a 'where' clause
+  = HsValBinds (HsValBinds id)
+  | HsIPBinds  (HsIPBinds id)
+  | EmptyLocalBinds
+
+data HsValBinds id     -- Value bindings (not implicit parameters)
+  = ValBindsIn                         -- Before typechecking
+       (LHsBinds id) [LSig id]         -- Not dependency analysed
+                                       -- Recursive by default
+
+  | ValBindsOut                                -- After renaming
+       [(RecFlag, LHsBinds id)]        -- Dependency analysed
+       [LSig Name]
+
+type LHsBinds id  = Bag (LHsBind id)
+type DictBinds id = LHsBinds id                -- Used for dictionary or method bindings
+type LHsBind  id  = Located (HsBind id)
+
+data HsBind id
+  = FunBind {  -- FunBind is used for both functions   f x = e
+               -- and variables                        f = \x -> e
+-- Reason 1: the Match stuff lets us have an optional
+--        result type sig      f :: a->a = ...mentions a...
+--
+-- Reason 2: Special case for type inference: see TcBinds.tcMonoBinds
+--
+-- Reason 3: instance decls can only have FunBinds, which is convenient
+--          If you change this, you'll need tochange e.g. rnMethodBinds
+
+       fun_id :: Located id,
+
+       fun_infix :: Bool,      -- True => infix declaration
+
+       fun_matches :: MatchGroup id,   -- The payload
+
+       fun_co_fn :: ExprCoFn,  -- Coercion from the type of the MatchGroup to the type of
+                               -- the Id.  Example:
+                               --      f :: Int -> forall a. a -> a
+                               --      f x y = y
+                               -- Then the MatchGroup will have type (Int -> a' -> a')
+                               -- (with a free type variable a').  The coercion will take
+                               -- a CoreExpr of this type and convert it to a CoreExpr of
+                               -- type         Int -> forall a'. a' -> a'
+                               -- Notice that the coercion captures the free a'.  That's
+                               -- why coercions are (CoreExpr -> CoreExpr), rather than
+                               -- just CoreExpr (with a functional type)
+
+       bind_fvs :: NameSet     -- After the renamer, this contains a superset of the 
+                               -- Names of the other binders in this binding group that 
+                               -- are free in the RHS of the defn
+                               -- Before renaming, and after typechecking, 
+                               -- the field is unused; it's just an error thunk
+    }
+
+  | PatBind {  -- The pattern is never a simple variable;
+               -- That case is done by FunBind
+       pat_lhs    :: LPat id,
+       pat_rhs    :: GRHSs id,
+       pat_rhs_ty :: PostTcType,       -- Type of the GRHSs
+       bind_fvs   :: NameSet           -- Same as for FunBind
+    }
+
+  | VarBind {  -- Dictionary binding and suchlike 
+       var_id :: id,           -- All VarBinds are introduced by the type checker
+       var_rhs :: LHsExpr id   -- Located only for consistency
+    }
+
+  | AbsBinds {                                 -- Binds abstraction; TRANSLATION
+       abs_tvs     :: [TyVar],  
+       abs_dicts   :: [DictId],
+       abs_exports :: [([TyVar], id, id, [Prag])],     -- (tvs, poly_id, mono_id, prags)
+       abs_binds   :: LHsBinds id              -- The dictionary bindings and typechecked user bindings
+                                               -- mixed up together; you can tell the dict bindings because
+                                               -- they are all VarBinds
+    }
+       -- Consider (AbsBinds tvs ds [(ftvs, poly_f, mono_f) binds]
+       -- 
+       -- Creates bindings for (polymorphic, overloaded) poly_f
+       -- in terms of monomorphic, non-overloaded mono_f
+       --
+       -- Invariants: 
+       --      1. 'binds' binds mono_f
+       --      2. ftvs is a subset of tvs
+       --      3. ftvs includes all tyvars free in ds
+       --
+       -- See section 9 of static semantics paper for more details.
+       -- (You can get a PhD for explaining the True Meaning
+       --  of this last construct.)
 
-\begin{code}
-nullBinds :: HsBinds id -> Bool
+placeHolderNames :: NameSet
+-- Used for the NameSet in FunBind and PatBind prior to the renamer
+placeHolderNames = panic "placeHolderNames"
+
+------------
+instance OutputableBndr id => Outputable (HsLocalBinds id) where
+  ppr (HsValBinds bs) = ppr bs
+  ppr (HsIPBinds bs)  = ppr bs
+  ppr EmptyLocalBinds = empty
+
+instance OutputableBndr id => Outputable (HsValBinds id) where
+  ppr (ValBindsIn binds sigs)
+   = pprValBindsForUser binds sigs
+
+  ppr (ValBindsOut sccs sigs) 
+    = getPprStyle $ \ sty ->
+      if debugStyle sty then   -- Print with sccs showing
+       vcat (map ppr sigs) $$ vcat (map ppr_scc sccs)
+     else
+       pprValBindsForUser (unionManyBags (map snd sccs)) sigs
+   where
+     ppr_scc (rec_flag, binds) = pp_rec rec_flag <+> pprLHsBinds binds
+     pp_rec Recursive    = ptext SLIT("rec")
+     pp_rec NonRecursive = ptext SLIT("nonrec")
+
+--  *not* pprLHsBinds because we don't want braces; 'let' and
+-- 'where' include a list of HsBindGroups and we don't want
+-- several groups of bindings each with braces around.
+-- Sort by location before printing
+pprValBindsForUser binds sigs
+  = vcat (map snd (sort_by_loc decls))
+  where
 
-nullBinds EmptyBinds           = True
-nullBinds (ThenBinds b1 b2)    = nullBinds b1 && nullBinds b2
-nullBinds (MonoBind b _ _)     = nullMonoBinds b
-nullBinds (IPBinds b _)        = null b
+    decls :: [(SrcSpan, SDoc)]
+    decls = [(loc, ppr sig)  | L loc sig <- sigs] ++
+           [(loc, ppr bind) | L loc bind <- bagToList binds]
 
-mkMonoBind :: RecFlag -> MonoBinds id -> HsBinds id
-mkMonoBind _      EmptyMonoBinds  = EmptyBinds
-mkMonoBind is_rec mbinds         = MonoBind mbinds [] is_rec
-\end{code}
+    sort_by_loc decls = sortLe (\(l1,_) (l2,_) -> l1 <= l2) decls
 
-\begin{code}
-instance (OutputableBndr id) => Outputable (HsBinds id) where
-    ppr binds = ppr_binds binds
+pprLHsBinds :: OutputableBndr id => LHsBinds id -> SDoc
+pprLHsBinds binds 
+  | isEmptyLHsBinds binds = empty
+  | otherwise = lbrace <+> vcat (map ppr (bagToList binds)) <+> rbrace
 
-ppr_binds EmptyBinds = empty
-ppr_binds (ThenBinds binds1 binds2)
-    = ppr_binds binds1 $$ ppr_binds binds2
+------------
+emptyLocalBinds :: HsLocalBinds a
+emptyLocalBinds = EmptyLocalBinds
 
-ppr_binds (IPBinds binds is_with)
-  = sep (punctuate semi (map pp_item binds))
-  where
-    pp_item (id,rhs) = pprBndr LetBind id <+> equals <+> pprExpr rhs
-
-ppr_binds (MonoBind bind sigs is_rec)
-     = vcat [ppr_isrec,
-            vcat (map ppr sigs),
-            ppr bind
-       ]
-     where
-       ppr_isrec = getPprStyle $ \ sty -> 
-                  if userStyle sty then empty else
-                  case is_rec of
-                       Recursive    -> ptext SLIT("{- rec -}")
-                       NonRecursive -> ptext SLIT("{- nonrec -}")
-\end{code}
+isEmptyLocalBinds :: HsLocalBinds a -> Bool
+isEmptyLocalBinds (HsValBinds ds) = isEmptyValBinds ds
+isEmptyLocalBinds (HsIPBinds ds)  = isEmptyIPBinds ds
+isEmptyLocalBinds EmptyLocalBinds = True
 
-%************************************************************************
-%*                                                                     *
-\subsection{Bindings: @MonoBinds@}
-%*                                                                     *
-%************************************************************************
+isEmptyValBinds :: HsValBinds a -> Bool
+isEmptyValBinds (ValBindsIn ds sigs)  = isEmptyLHsBinds ds && null sigs
+isEmptyValBinds (ValBindsOut ds sigs) = null ds && null sigs
 
-Global bindings (where clauses)
+emptyValBindsIn, emptyValBindsOut :: HsValBinds a
+emptyValBindsIn  = ValBindsIn emptyBag []
+emptyValBindsOut = ValBindsOut []      []
 
-\begin{code}
-data MonoBinds id
-  = EmptyMonoBinds
-
-  | AndMonoBinds    (MonoBinds id)
-                   (MonoBinds id)
-
-  | FunMonoBind     id         -- Used for both functions      f x = e
-                               -- and variables                f = \x -> e
-                               -- Reason: the Match stuff lets us have an optional
-                               --         result type sig      f :: a->a = ...mentions a...
-                               --
-                               -- This also means that instance decls can only have
-                               -- FunMonoBinds, so if you change this, you'll need to
-                               -- change e.g. rnMethodBinds
-                   Bool                -- True => infix declaration
-                   [Match id]
-                   SrcLoc
-
-  | PatMonoBind     (Pat id)   -- The pattern is never a simple variable;
-                               -- That case is done by FunMonoBind
-                   (GRHSs id)
-                   SrcLoc
-
-  | VarMonoBind            id                  -- TRANSLATION
-                   (HsExpr id)
-
-  | AbsBinds                           -- Binds abstraction; TRANSLATION
-               [TyVar]                 -- Type variables
-               [id]                    -- Dicts
-               [([TyVar], id, id)]     -- (type variables, polymorphic, momonmorphic) triples
-               NameSet                 -- Set of *polymorphic* variables that have an INLINE pragma
-               (MonoBinds id)      -- The "business end"
-
-       -- Creates bindings for *new* (polymorphic, overloaded) locals
-       -- in terms of *old* (monomorphic, non-overloaded) ones.
-       --
-       -- See section 9 of static semantics paper for more details.
-       -- (You can get a PhD for explaining the True Meaning
-       --  of this last construct.)
+emptyLHsBinds :: LHsBinds id
+emptyLHsBinds = emptyBag
+
+isEmptyLHsBinds :: LHsBinds id -> Bool
+isEmptyLHsBinds = isEmptyBag
+
+------------
+plusHsValBinds :: HsValBinds a -> HsValBinds a -> HsValBinds a
+plusHsValBinds (ValBindsIn ds1 sigs1) (ValBindsIn ds2 sigs2)
+  = ValBindsIn (ds1 `unionBags` ds2) (sigs1 ++ sigs2)
+plusHsValBinds (ValBindsOut ds1 sigs1) (ValBindsOut ds2 sigs2)
+  = ValBindsOut (ds1 ++ ds2) (sigs1 ++ sigs2)
 \end{code}
 
 What AbsBinds means
@@ -176,63 +226,96 @@ So the desugarer tries to do a better job:
                                       in (fm,gm)
 
 \begin{code}
--- We keep the invariant that a MonoBinds is only empty 
--- if it is exactly EmptyMonoBinds
+instance OutputableBndr id => Outputable (HsBind id) where
+    ppr mbind = ppr_monobind mbind
 
-nullMonoBinds :: MonoBinds id -> Bool
-nullMonoBinds EmptyMonoBinds        = True
-nullMonoBinds other_monobind        = False
+ppr_monobind :: OutputableBndr id => HsBind id -> SDoc
 
-andMonoBinds :: MonoBinds id -> MonoBinds id -> MonoBinds id
-andMonoBinds EmptyMonoBinds mb = mb
-andMonoBinds mb EmptyMonoBinds = mb
-andMonoBinds mb1 mb2 = AndMonoBinds mb1 mb2
+ppr_monobind (PatBind { pat_lhs = pat, pat_rhs = grhss })      = pprPatBind pat grhss
+ppr_monobind (VarBind { var_id = var, var_rhs = rhs })         = ppr var <+> equals <+> pprExpr (unLoc rhs)
+ppr_monobind (FunBind { fun_id = fun, fun_matches = matches }) = pprFunBind (unLoc fun) matches
+      -- ToDo: print infix if appropriate
 
-andMonoBindList :: [MonoBinds id] -> MonoBinds id
-andMonoBindList binds
-  = loop1 binds
+ppr_monobind (AbsBinds { abs_tvs = tyvars, abs_dicts = dictvars, 
+                        abs_exports = exports, abs_binds = val_binds })
+     = sep [ptext SLIT("AbsBinds"),
+           brackets (interpp'SP tyvars),
+           brackets (interpp'SP dictvars),
+           brackets (sep (punctuate comma (map ppr_exp exports)))]
+       $$
+       nest 2 ( vcat [pprBndr LetBind x | (_,x,_,_) <- exports]
+                       -- Print type signatures
+               $$ pprLHsBinds val_binds )
   where
-    loop1 [] = EmptyMonoBinds
-    loop1 (EmptyMonoBinds : binds) = loop1 binds
-    loop1 (b:bs) = loop2 b bs
-
-       -- acc is non-empty
-    loop2 acc [] = acc
-    loop2 acc (EmptyMonoBinds : bs) = loop2 acc bs
-    loop2 acc (b:bs) = loop2 (acc `AndMonoBinds` b) bs
+    ppr_exp (tvs, gbl, lcl, prags)
+       = vcat [ppr gbl <+> ptext SLIT("<=") <+> ppr tvs <+> ppr lcl,
+               nest 2 (vcat (map (pprPrag gbl) prags))]
 \end{code}
 
+%************************************************************************
+%*                                                                     *
+               Implicit parameter bindings
+%*                                                                     *
+%************************************************************************
 
 \begin{code}
-instance OutputableBndr id => Outputable (MonoBinds id) where
-    ppr mbind = ppr_monobind mbind
+data HsIPBinds id
+  = IPBinds 
+       [LIPBind id] 
+       (DictBinds id)  -- Only in typechecker output; binds 
+                       -- uses of the implicit parameters
 
+isEmptyIPBinds :: HsIPBinds id -> Bool
+isEmptyIPBinds (IPBinds is ds) = null is && isEmptyBag ds
 
-ppr_monobind :: OutputableBndr id => MonoBinds id -> SDoc
-ppr_monobind EmptyMonoBinds = empty
-ppr_monobind (AndMonoBinds binds1 binds2)
-      = ppr_monobind binds1 $$ ppr_monobind binds2
+type LIPBind id = Located (IPBind id)
 
-ppr_monobind (PatMonoBind pat grhss locn)      = pprPatBind pat grhss
-ppr_monobind (FunMonoBind fun inf matches locn) = pprFunBind fun matches
-      -- ToDo: print infix if appropriate
+-- | Implicit parameter bindings.
+data IPBind id
+  = IPBind
+       (IPName id)
+       (LHsExpr id)
 
-ppr_monobind (VarMonoBind name expr)
-      = sep [pprBndr LetBind name <+> equals, nest 4 (pprExpr expr)]
+instance (OutputableBndr id) => Outputable (HsIPBinds id) where
+  ppr (IPBinds bs ds) = vcat (map ppr bs) 
+                       $$ pprLHsBinds ds
 
-ppr_monobind (AbsBinds tyvars dictvars exports inlines val_binds)
-     = sep [ptext SLIT("AbsBinds"),
-           brackets (interpp'SP tyvars),
-           brackets (interpp'SP dictvars),
-           brackets (sep (punctuate comma (map ppr exports))),
-           brackets (interpp'SP (nameSetToList inlines))]
-       $$
-       nest 4 ( vcat [pprBndr LetBind x | (_,x,_) <- exports]
-                       -- Print type signatures
-               $$
-               ppr val_binds )
+instance (OutputableBndr id) => Outputable (IPBind id) where
+  ppr (IPBind id rhs) = pprBndr LetBind id <+> equals <+> pprExpr (unLoc rhs)
 \end{code}
 
+
+%************************************************************************
+%*                                                                     *
+\subsection{Coercion functions}
+%*                                                                     *
+%************************************************************************
+
+\begin{code}
+-- A Coercion is an expression with a hole in it
+-- We need coercions to have concrete form so that we can zonk them
+
+data ExprCoFn
+  = CoHole                     -- The identity coercion
+  | CoCompose ExprCoFn ExprCoFn
+  | CoApps ExprCoFn [Id]               -- Non-empty list
+  | CoTyApps ExprCoFn [Type]           --   in all of these
+  | CoLams [Id] ExprCoFn               --   so that the identity coercion
+  | CoTyLams [TyVar] ExprCoFn          --   is just Hole
+  | CoLet (LHsBinds Id) ExprCoFn       -- Would be nicer to be core bindings
+
+(<.>) :: ExprCoFn -> ExprCoFn -> ExprCoFn
+(<.>) = CoCompose
+
+idCoercion :: ExprCoFn
+idCoercion = CoHole
+
+isIdCoercion :: ExprCoFn -> Bool
+isIdCoercion CoHole = True
+isIdCoercion other  = False
+\end{code}
+
+
 %************************************************************************
 %*                                                                     *
 \subsection{@Sig@: type signatures and value-modifying user pragmas}
@@ -245,141 +328,151 @@ signatures.  Then all the machinery to move them into place, etc.,
 serves for both.
 
 \begin{code}
+type LSig name = Located (Sig name)
+
 data Sig name
-  = Sig                name            -- a bog-std type signature
-               (HsType name)
-               SrcLoc
-
-  | ClassOpSig name            -- Selector name
-                (DefMeth name) -- Default-method info
-                               -- See "THE NAMING STORY" in HsDecls
-               (HsType name)
-               SrcLoc
-
-  | SpecSig    name            -- specialise a function or datatype ...
-               (HsType name)   -- ... to these types
-               SrcLoc
-
-  | InlineSig  Bool            -- True <=> INLINE f, False <=> NOINLINE f
-               name            -- Function name
-               Activation      -- When inlining is *active*
-               SrcLoc
-
-  | SpecInstSig (HsType name)  -- (Class tys); should be a specialisation of the 
+  = TypeSig    (Located name)  -- A bog-std type signature
+               (LHsType name)
+
+  | SpecSig    (Located name)  -- Specialise a function or datatype ...
+               (LHsType name)  -- ... to these types
+               InlineSpec
+
+  | InlineSig  (Located name)  -- Function name
+               InlineSpec
+
+  | SpecInstSig (LHsType name) -- (Class tys); should be a specialisation of the 
                                -- current instance decl
-               SrcLoc
 
   | FixSig     (FixitySig name)        -- Fixity declaration
+
+type LFixitySig name = Located (FixitySig name)
+data FixitySig name = FixitySig (Located name) Fixity 
+
+-- A Prag conveys pragmas from the type checker to the desugarer
+data Prag 
+  = InlinePrag 
+       InlineSpec
+
+  | SpecPrag   
+       (HsExpr Id)     -- An expression, of the given specialised type, which
+       PostTcType      -- specialises the polymorphic function
+       [Id]            -- Dicts mentioned free in the expression
+       InlineSpec      -- Inlining spec for the specialised function
+
+isInlinePrag (InlinePrag _) = True
+isInlinePrag prag          = False
+
+isSpecPrag (SpecPrag _ _ _ _) = True
+isSpecPrag prag                      = False
 \end{code}
 
 \begin{code}
-okBindSig :: NameSet -> Sig Name -> Bool
-okBindSig ns (ClassOpSig _ _ _ _) = False
-okBindSig ns sig                 = sigForThisGroup ns sig
-
-okClsDclSig :: Sig Name -> Bool
-okClsDclSig (Sig _ _ _)       = False
-okClsDclSig (SpecInstSig _ _) = False
-okClsDclSig sig              = True    -- All others OK
-
-okInstDclSig :: NameSet -> Sig Name -> Bool
-okInstDclSig ns (Sig _ _ _)      = False
-okInstDclSig ns (FixSig _)       = False
-okInstDclSig ns (SpecInstSig _ _) = True
-okInstDclSig ns sig              = sigForThisGroup ns sig
-
-sigForThisGroup ns sig 
+okBindSig :: NameSet -> LSig Name -> Bool
+okBindSig ns sig = sigForThisGroup ns sig
+
+okHsBootSig :: LSig Name -> Bool
+okHsBootSig (L _ (TypeSig  _ _)) = True
+okHsBootSig (L _ (FixSig _))    = True
+okHsBootSig sig                         = False
+
+okClsDclSig :: LSig Name -> Bool
+okClsDclSig (L _ (SpecInstSig _)) = False
+okClsDclSig sig                  = True        -- All others OK
+
+okInstDclSig :: NameSet -> LSig Name -> Bool
+okInstDclSig ns lsig@(L _ sig) = ok ns sig
+  where
+    ok ns (TypeSig _ _)          = False
+    ok ns (FixSig _)     = False
+    ok ns (SpecInstSig _) = True
+    ok ns sig            = sigForThisGroup ns lsig
+
+sigForThisGroup :: NameSet -> LSig Name -> Bool
+sigForThisGroup ns sig
   = case sigName sig of
-       Nothing                  -> False
-       Just n | isUnboundName n -> True        -- Don't complain about an unbound name again
-              | otherwise       -> n `elemNameSet` ns
-
-sigName :: Sig name -> Maybe name
-sigName (Sig         n _ _)        = Just n
-sigName (ClassOpSig  n _ _ _)      = Just n
-sigName (SpecSig     n _ _)        = Just n
-sigName (InlineSig _ n _ _)        = Just n
-sigName (FixSig (FixitySig n _ _)) = Just n
-sigName other                     = Nothing
-
-isFixitySig :: Sig name -> Bool
-isFixitySig (FixSig _) = True
-isFixitySig _         = False
-
-isClassOpSig :: Sig name -> Bool
-isClassOpSig (ClassOpSig _ _ _ _) = True
-isClassOpSig _                   = False
-
-isPragSig :: Sig name -> Bool
+       Nothing -> False
+       Just n  -> n `elemNameSet` ns
+
+sigName :: LSig name -> Maybe name
+sigName (L _ sig) = f sig
+ where
+    f (TypeSig   n _)          = Just (unLoc n)
+    f (SpecSig   n _ _)        = Just (unLoc n)
+    f (InlineSig n _)          = Just (unLoc n)
+    f (FixSig (FixitySig n _)) = Just (unLoc n)
+    f other                    = Nothing
+
+isFixityLSig :: LSig name -> Bool
+isFixityLSig (L _ (FixSig {})) = True
+isFixityLSig _                = False
+
+isVanillaLSig :: LSig name -> Bool
+isVanillaLSig (L _(TypeSig {})) = True
+isVanillaLSig sig              = False
+
+isSpecLSig :: LSig name -> Bool
+isSpecLSig (L _(SpecSig {})) = True
+isSpecLSig sig              = False
+
+isSpecInstLSig (L _ (SpecInstSig {})) = True
+isSpecInstLSig sig                   = False
+
+isPragLSig :: LSig name -> Bool
        -- Identifies pragmas 
-isPragSig (SpecSig _ _ _)     = True
-isPragSig (InlineSig _ _ _ _) = True
-isPragSig (SpecInstSig _ _)   = True
-isPragSig other                      = False
+isPragLSig (L _ (SpecSig {}))   = True
+isPragLSig (L _ (InlineSig {})) = True
+isPragLSig other               = False
+
+isInlineLSig :: LSig name -> Bool
+       -- Identifies inline pragmas 
+isInlineLSig (L _ (InlineSig {})) = True
+isInlineLSig other               = False
+
+hsSigDoc (TypeSig {})          = ptext SLIT("type signature")
+hsSigDoc (SpecSig {})          = ptext SLIT("SPECIALISE pragma")
+hsSigDoc (InlineSig _ spec)    = ppr spec <+> ptext SLIT("pragma")
+hsSigDoc (SpecInstSig {})      = ptext SLIT("SPECIALISE instance pragma")
+hsSigDoc (FixSig {})           = ptext SLIT("fixity declaration")
 \end{code}
 
+Signature equality is used when checking for duplicate signatures
+
 \begin{code}
-hsSigDoc (Sig        _ _ loc)        = (ptext SLIT("type signature"),loc)
-hsSigDoc (ClassOpSig _ _ _ loc)       = (ptext SLIT("class-method type signature"), loc)
-hsSigDoc (SpecSig    _ _ loc)        = (ptext SLIT("SPECIALISE pragma"),loc)
-hsSigDoc (InlineSig True  _ _ loc)    = (ptext SLIT("INLINE pragma"),loc)
-hsSigDoc (InlineSig False _ _ loc)    = (ptext SLIT("NOINLINE pragma"),loc)
-hsSigDoc (SpecInstSig _ loc)         = (ptext SLIT("SPECIALISE instance pragma"),loc)
-hsSigDoc (FixSig (FixitySig _ _ loc)) = (ptext SLIT("fixity declaration"), loc)
+eqHsSig :: LSig Name -> LSig Name -> Bool
+eqHsSig (L _ (FixSig (FixitySig n1 _))) (L _ (FixSig (FixitySig n2 _))) = unLoc n1 == unLoc n2
+eqHsSig (L _ (TypeSig n1 _))           (L _ (TypeSig n2 _))            = unLoc n1 == unLoc n2
+eqHsSig (L _ (InlineSig n1 s1))        (L _ (InlineSig n2 s2))                 = s1 == s2 && unLoc n1 == unLoc n2
+       -- For specialisations, we don't have equality over
+       -- HsType, so it's not convenient to spot duplicate 
+       -- specialisations here.  Check for this later, when we're in Type land
+eqHsSig _other1 _other2 = False
 \end{code}
 
 \begin{code}
-instance (Outputable name) => Outputable (Sig name) where
+instance (OutputableBndr name) => Outputable (Sig name) where
     ppr sig = ppr_sig sig
 
-ppr_sig :: Outputable name => Sig name -> SDoc
-ppr_sig (Sig var ty _)
-      = sep [ppr var <+> dcolon, nest 4 (ppr ty)]
-
-ppr_sig (ClassOpSig var dm ty _)
-      = sep [ pprHsVar var <+> dcolon, 
-             nest 4 (ppr ty),
-             nest 4 (pp_dm_comment) ]
-      where
-       pp_dm = case dm of 
-                 DefMeth _  -> equals  -- Default method indicator
-                 GenDefMeth -> semi    -- Generic method indicator
-                 NoDefMeth  -> empty   -- No Method at all
-       pp_dm_comment = case dm of 
-                 DefMeth _  -> text "{- has default method -}"
-                 GenDefMeth -> text "{- has generic method -}"
-                 NoDefMeth  -> empty   -- No Method at all
-
-ppr_sig (SpecSig var ty _)
-      = sep [ hsep [text "{-# SPECIALIZE", ppr var, dcolon],
-             nest 4 (ppr ty <+> text "#-}")
-       ]
-
-ppr_sig (InlineSig True var phase _)
-      = hsep [text "{-# INLINE", ppr phase, ppr var, text "#-}"]
-
-ppr_sig (InlineSig False var phase _)
-      = hsep [text "{-# NOINLINE", ppr phase, ppr var, text "#-}"]
-
-ppr_sig (SpecInstSig ty _)
-      = hsep [text "{-# SPECIALIZE instance", ppr ty, text "#-}"]
-
-ppr_sig (FixSig fix_sig) = ppr fix_sig
-\end{code}
+ppr_sig :: OutputableBndr name => Sig name -> SDoc
+ppr_sig (TypeSig var ty)         = pprVarSig (unLoc var) ty
+ppr_sig (FixSig fix_sig)         = ppr fix_sig
+ppr_sig (SpecSig var ty inl)     = pragBrackets (pprSpec var ty inl)
+ppr_sig (InlineSig var inl)       = pragBrackets (ppr inl <+> ppr var)
+ppr_sig (SpecInstSig ty)         = pragBrackets (ptext SLIT("SPECIALIZE instance") <+> ppr ty)
 
-Checking for distinct signatures; oh, so boring
+instance Outputable name => Outputable (FixitySig name) where
+  ppr (FixitySig name fixity) = sep [ppr fixity, ppr name]
 
+pragBrackets :: SDoc -> SDoc
+pragBrackets doc = ptext SLIT("{-#") <+> doc <+> ptext SLIT("#-}") 
 
-\begin{code}
-eqHsSig :: Sig Name -> Sig Name -> Bool
-eqHsSig (Sig n1 _ _)         (Sig n2 _ _)          = n1 == n2
-eqHsSig (InlineSig b1 n1 _ _)(InlineSig b2 n2 _ _) = b1 == b2 && n1 == n2
+pprVarSig :: (Outputable id, Outputable ty) => id -> ty -> SDoc
+pprVarSig var ty = sep [ppr var <+> dcolon, nest 2 (ppr ty)]
 
-eqHsSig (SpecInstSig ty1 _)  (SpecInstSig ty2 _)  = ty1 == ty2
-eqHsSig (SpecSig n1 ty1 _)   (SpecSig n2 ty2 _)   =
-    -- may have many specialisations for one value;
-    -- but not ones that are exactly the same...
-    (n1 == n2) && (ty1 == ty2)
+pprSpec :: (Outputable id, Outputable ty) => id -> ty -> InlineSpec -> SDoc
+pprSpec var ty inl = sep [ptext SLIT("SPECIALIZE") <+> ppr inl <+> pprVarSig var ty]
 
-eqHsSig _other1 _other2 = False
+pprPrag :: Outputable id => id -> Prag -> SDoc
+pprPrag var (InlinePrag inl)         = ppr inl <+> ppr var
+pprPrag var (SpecPrag expr ty _ inl) = pprSpec var ty inl
 \end{code}