Add bang patterns
[ghc-hetmet.git] / ghc / compiler / hsSyn / HsBinds.lhs
index 9576c6d..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 ( pprExpr, HsExpr, pprMatches, Match, pprGRHSs, GRHSs )
+import {-# SOURCE #-} HsExpr ( HsExpr, pprExpr, LHsExpr,
+                              MatchGroup, pprFunBind,
+                              GRHSs, pprPatBind )
+import {-# SOURCE #-} HsPat  ( LPat )
 
--- friends:
-import HsTypes         ( HsType )
-import CoreSyn         ( CoreExpr )
-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(..), Fixity )
+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 pat            -- binders and bindees
-  = EmptyBinds
+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.)
 
-  | ThenBinds  (HsBinds id pat)
-               (HsBinds id pat)
+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
 
-  | MonoBind   (MonoBinds id pat)
-               [Sig id]                -- Empty on typechecker output
-               RecFlag
-\end{code}
+    decls :: [(SrcSpan, SDoc)]
+    decls = [(loc, ppr sig)  | L loc sig <- sigs] ++
+           [(loc, ppr bind) | L loc bind <- bagToList binds]
 
-\begin{code}
-nullBinds :: HsBinds id pat -> Bool
+    sort_by_loc decls = sortLe (\(l1,_) (l2,_) -> l1 <= l2) decls
 
-nullBinds EmptyBinds           = True
-nullBinds (ThenBinds b1 b2)    = nullBinds b1 && nullBinds b2
-nullBinds (MonoBind b _ _)     = nullMonoBinds b
+pprLHsBinds :: OutputableBndr id => LHsBinds id -> SDoc
+pprLHsBinds binds 
+  | isEmptyLHsBinds binds = empty
+  | otherwise = lbrace <+> vcat (map ppr (bagToList binds)) <+> rbrace
 
-mkMonoBind :: MonoBinds id pat -> [Sig id] -> RecFlag -> HsBinds id pat
-mkMonoBind EmptyMonoBinds _ _ = EmptyBinds
-mkMonoBind mbinds sigs is_rec = MonoBind mbinds sigs is_rec
-\end{code}
+------------
+emptyLocalBinds :: HsLocalBinds a
+emptyLocalBinds = EmptyLocalBinds
 
-\begin{code}
-instance (Outputable pat, Outputable id) =>
-               Outputable (HsBinds id pat) where
-    ppr binds = ppr_binds binds
-
-ppr_binds EmptyBinds = empty
-ppr_binds (ThenBinds binds1 binds2)
-    = ppr_binds binds1 $$ ppr_binds binds2
-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 pat
-  = EmptyMonoBinds
-
-  | AndMonoBinds    (MonoBinds id pat)
-                   (MonoBinds id pat)
-
-  | 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...
-                   Bool                -- True => infix declaration
-                   [Match id pat]
-                   SrcLoc
-
-  | PatMonoBind     pat                -- The pattern is never a simple variable;
-                               -- That case is done by FunMonoBind
-                   (GRHSs id pat)
-                   SrcLoc
-
-  | VarMonoBind            id                  -- TRANSLATION
-                   (HsExpr id pat)
-
-  | CoreMonoBind    id                 -- TRANSLATION
-                   CoreExpr            -- No zonking; this is a final CoreExpr with Ids and Types!
-
-  | 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 pat)      -- 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
@@ -157,71 +222,100 @@ So the desugarer tries to do a better job:
                                        (fm,gm) -> fm
        ..ditto for gp..
 
-       p = /\ [a,b] -> \ [d1,d2] -> letrec DBINDS and BIND 
-                                     in (fm,gm)
+       tp = /\ [a,b] -> \ [d1,d2] -> letrec DBINDS and BIND 
+                                      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 pat -> Bool
-nullMonoBinds EmptyMonoBinds        = True
-nullMonoBinds other_monobind        = False
+ppr_monobind :: OutputableBndr id => HsBind id -> SDoc
 
-andMonoBinds :: MonoBinds id pat -> MonoBinds id pat -> MonoBinds id pat
-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 pat] -> MonoBinds id pat
-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 (Outputable id, Outputable pat) =>
-               Outputable (MonoBinds id pat) 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 :: (Outputable id, Outputable pat) => MonoBinds id pat -> 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)
-      = sep [ppr pat, nest 4 (pprGRHSs False grhss)]
+-- | Implicit parameter bindings.
+data IPBind id
+  = IPBind
+       (IPName id)
+       (LHsExpr id)
 
-ppr_monobind (FunMonoBind fun inf matches locn)
-      = pprMatches (False, ppr fun) matches
-      -- ToDo: print infix if appropriate
+instance (OutputableBndr id) => Outputable (HsIPBinds id) where
+  ppr (IPBinds bs ds) = vcat (map ppr bs) 
+                       $$ pprLHsBinds ds
 
-ppr_monobind (VarMonoBind name expr)
-      = sep [ppr name <+> equals, nest 4 (pprExpr expr)]
+instance (OutputableBndr id) => Outputable (IPBind id) where
+  ppr (IPBind id rhs) = pprBndr LetBind id <+> equals <+> pprExpr (unLoc rhs)
+\end{code}
 
-ppr_monobind (CoreMonoBind name expr)
-      = sep [ppr name <+> equals, nest 4 (ppr expr)]
 
-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 (ppr val_binds)
+%************************************************************************
+%*                                                                     *
+\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}
@@ -234,154 +328,151 @@ signatures.  Then all the machinery to move them into place, etc.,
 serves for both.
 
 \begin{code}
-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
+type LSig name = Located (Sig name)
 
-  | SpecSig    name            -- specialise a function or datatype ...
-               (HsType name)   -- ... to these types
-               SrcLoc
+data Sig name
+  = TypeSig    (Located name)  -- A bog-std type signature
+               (LHsType name)
 
-  | InlineSig  name            -- INLINE f
-               (Maybe Int)     -- phase
-               SrcLoc
+  | SpecSig    (Located name)  -- Specialise a function or datatype ...
+               (LHsType name)  -- ... to these types
+               InlineSpec
 
-  | NoInlineSig        name            -- NOINLINE f
-               (Maybe Int)     -- phase
-               SrcLoc
+  | InlineSig  (Located name)  -- Function name
+               InlineSpec
 
-  | SpecInstSig (HsType name)  -- (Class tys); should be a specialisation of the 
+  | 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
 
-data FixitySig name = FixitySig name Fixity SrcLoc 
+  | 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
 
-instance Eq name => Eq (FixitySig name) where
-   (FixitySig n1 f1 _) == (FixitySig n2 f2 _) = n1==n2 && f1==f2
+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 :: NameSet -> LSig Name -> Bool
 okBindSig ns sig = sigForThisGroup ns sig
 
-okClsDclSig :: NameSet -> Sig Name -> Bool
-okClsDclSig ns (Sig _ _ _)                                       = False
-okClsDclSig 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 -> Sig Name -> Bool
-okInstDclSig ns (Sig _ _ _)                                       = False
-okInstDclSig ns (FixSig _)                                        = False
-okInstDclSig ns (SpecInstSig _ _)                                 = True
-okInstDclSig ns sig = sigForThisGroup ns sig
+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 ns sig 
+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 (NoInlineSig 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 (NoInlineSig _ _ _) = 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)        = (SLIT("type signature"),loc)
-hsSigDoc (ClassOpSig _ _ _ loc)       = (SLIT("class-method type signature"), loc)
-hsSigDoc (SpecSig    _ _ loc)        = (SLIT("SPECIALISE pragma"),loc)
-hsSigDoc (InlineSig  _ _    loc)      = (SLIT("INLINE pragma"),loc)
-hsSigDoc (NoInlineSig  _ _  loc)      = (SLIT("NOINLINE pragma"),loc)
-hsSigDoc (SpecInstSig _ loc)         = (SLIT("SPECIALISE instance pragma"),loc)
-hsSigDoc (FixSig (FixitySig _ _ loc)) = (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 [ppr var <+> pp_dm <+> dcolon, nest 4 (ppr ty)]
-      where
-       pp_dm = case dm of 
-                 DefMeth _  -> equals  -- Default method indicator
-                 GenDefMeth -> semi    -- Generic method indicator
-                 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 var phase _)
-      = hsep [text "{-# INLINE", ppr_phase phase, ppr var, text "#-}"]
-
-ppr_sig (NoInlineSig var phase _)
-      = hsep [text "{-# NOINLINE", ppr_phase phase, ppr var, text "#-}"]
-
-ppr_sig (SpecInstSig ty _)
-      = hsep [text "{-# SPECIALIZE instance", ppr ty, text "#-}"]
-
-ppr_sig (FixSig fix_sig) = ppr fix_sig
-
+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)
 
 instance Outputable name => Outputable (FixitySig name) where
-  ppr (FixitySig name fixity loc) = sep [ppr fixity, ppr name]
-
-ppr_phase :: Maybe Int -> SDoc
-ppr_phase Nothing  = empty
-ppr_phase (Just n) = int n
-\end{code}
-
-Checking for distinct signatures; oh, so boring
+  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 n1 _ _)   (InlineSig n2 _ _)   = n1 == n2
-eqHsSig (NoInlineSig n1 _ _) (NoInlineSig n2 _ _) = 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}