[project @ 2000-10-13 14:14:31 by sewardj]
[ghc-hetmet.git] / ghc / compiler / main / MkIface.lhs
index 1e60923..1d709ef 100644 (file)
 %
-% (c) The GRASP/AQUA Project, Glasgow University, 1993-1995
+% (c) The GRASP/AQUA Project, Glasgow University, 1993-1998
 %
 \section[MkIface]{Print an interface for a module}
 
 \begin{code}
+module MkIface ( writeIface  ) where
+
 #include "HsVersions.h"
 
-module MkIface ( mkInterface ) where
+import IO              ( openFile, hClose, IOMode(..) )
+
+import HsSyn
+import HsCore          ( HsIdInfo(..), toUfExpr )
+import RdrHsSyn                ( RdrNameRuleDecl, mkTyData )
+import HsPragmas       ( DataPragmas(..), ClassPragmas(..) )
+import HsTypes         ( toHsTyVars )
+import BasicTypes      ( Fixity(..), NewOrData(..),
+                         Version, bumpVersion, initialVersion, isLoopBreaker
+                       )
+import RnMonad
+
+import TcInstUtil      ( InstInfo(..) )
 
-import PrelInfo                ( mkLiftTy, pRELUDE_CORE, pRELUDE_BUILTIN )
-import HsSyn           ( FixityDecl(..), RenamedFixityDecl(..), MonoBinds,
-                         RenamedMonoBinds(..), Name, RenamedPat(..), Sig
+import CmdLineOpts
+import Id              ( Id, idType, idInfo, omitIfaceSigForId, isUserExportedId, hasNoBinding,
+                         idSpecialisation
                        )
-import Type
+import Var             ( isId, varName )
+import VarSet
+import DataCon         ( StrictnessMark(..), dataConSig, dataConFieldLabels, dataConStrictMarks )
+import IdInfo          ( IdInfo, StrictnessInfo(..), ArityInfo(..), 
+                         CprInfo(..), CafInfo(..),
+                         inlinePragInfo, arityInfo, arityLowerBound,
+                         strictnessInfo, isBottomingStrictness,
+                         cafInfo, specInfo, cprInfo, 
+                         occInfo, isNeverInlinePrag,
+                         workerInfo, WorkerInfo(..)
+                       )
+import CoreSyn         ( CoreExpr, CoreBind, Bind(..), isBuiltinRule, rulesRules, rulesRhsFreeVars )
+import CoreFVs         ( exprSomeFreeVars, ruleSomeLhsFreeVars, ruleSomeFreeVars )
+import CoreUnfold      ( okToUnfoldInHiFile, couldBeSmallEnoughToInline )
+import Module          ( pprModuleName, moduleUserString )
+import Name            ( isLocallyDefined, isWiredInName, toRdrName, nameModule,
+                         Name, NamedThing(..)
+                       )
+import OccName         ( OccName, pprOccName )
+import TyCon           ( TyCon, getSynTyConDefn, isSynTyCon, isNewTyCon, isAlgTyCon,
+                         tyConTheta, tyConTyVars, tyConDataCons, tyConFamilySize
+                       )
+import Class           ( classExtraBigSig, DefMeth(..) )
+import FieldLabel      ( fieldLabelType )
+import Type            ( mkSigmaTy, splitSigmaTy, mkDictTy, tidyTopType,
+                         deNoteType, classesToPreds
+                       )
+
+import Rules           ( ProtoCoreRule(..) )
+
+import Bag             ( bagToList )
+import UniqFM          ( lookupUFM, listToUFM )
+import Util            ( sortLt )
+import SrcLoc          ( noSrcLoc )
 import Bag
-import FiniteMap
-import Id
-import IdInfo          -- plenty from here
-import Maybes          ( catMaybes, Maybe(..) )
 import Outputable
-import Pretty
-import StgSyn
-import TcInstDcls      ( InstInfo(..) )
-import Util
+import ErrUtils                ( dumpIfSet )
+
+import Maybe           ( isNothing )
+import List            ( partition )
+import Monad           ( when )
 \end{code}
 
+
 %************************************************************************
-%*                                                                     *
-\subsection[main-MkIface]{Main routine for making interfaces}
-%*                                                                     *
+%*                                                                     *
+\subsection{Write a new interface file}
+%*                                                                     *
 %************************************************************************
 
-Misc points:
-\begin{enumerate}
-\item
-We get the general what-to-export information from the ``environments''
-produced by the typechecker (the \tr{[RenamedFixityDecl]} through
-\tr{Bag InstInfo} arguments).
-
-\item
-{\em However:} Whereas (for example) an \tr{InstInfo} will have
-\tr{Ids} in it that identify the constant methods for that instance,
-those particular \tr{Ids} {\em do not have} the best @IdInfos@!!!
-Those @IdInfos@ were figured out long after the \tr{InstInfo} was
-created.
-
-That's why we actually look at the final \tr{StgBindings} that go
-into the code-generator: they have the best @IdInfos@ on them.
-Whenever, we are about to print info about an @Id@, we look in the
-Ids-from-STG-bindings list to see if we have an ``equivalent'' @Id@
-with presumably-better @IdInfo@.
-
-\item
-We play this same game whether for values, classes (for their
-method-selectors and default-methods), or instances (for their
-@DictFunIds@ or constant-methods).
-
-Of course, for imported things, what we got from the typechecker is
-all we're gonna get.
-
-\item
-We {\em sort} things in the interface into some ``canonical'' order;
-otherwise, with heavily-recursive modules, you can have (unchanged)
-information ``move around'' in the interface file---deeply unfriendly
-to \tr{make}.
-\end{enumerate}
-
 \begin{code}
-mkInterface :: FAST_STRING
-           -> (FAST_STRING -> Bool,  -- is something in export list, explicitly?
-               FAST_STRING -> Bool)  -- is a module among the "dotdot" exported modules?
-           -> IdEnv UnfoldingDetails
-           -> FiniteMap TyCon [(Bool, [Maybe Type])]
-           -> ([RenamedFixityDecl],  -- interface info from the typecheck
-               [Id],
-               CE,
-               TCE,
-               Bag InstInfo)
-           -> [StgBinding]
-           -> Pretty
-
-mkInterface modname export_list_fns inline_env tycon_specs
-           (fixity_decls, global_ids, ce, tce, inst_infos)
-           stg_binds
-  = let
-       -- first, gather up the things we want to export:
-
-       exported_tycons  = [ tc | tc <- rngTCE tce,
-                          isExported tc,
-                          is_exportable_tycon_or_class export_list_fns tc ]
-       exported_classes = [  c |  c <- rngCE  ce,
-                          isExported  c,
-                          is_exportable_tycon_or_class export_list_fns  c ]
-       exported_inst_infos = [ i | i <- bagToList inst_infos,
-                          is_exported_inst_info export_list_fns i ]
-       exported_vals
-         = [ v | v <- global_ids,
-             isExported v && not (isDataCon v) && not (isClassOpId v) ]
-
-       -- We also have to worry about TyCons/Classes that are
-       -- *mentioned* in exported things (e.g., values' types or
-       -- instances), so that we can be sure to do an import decl for
-       -- them, for original-naming purposes:
-
-       (mentioned_tycons, mentioned_classes)
-         = foldr ( \ (tcs1, cls1) (tcs2, cls2)
-                     -> (tcs1 `unionBags` tcs2, cls1 `unionBags` cls2) )
-                 (emptyBag, emptyBag)
-                 (map getMentionedTyConsAndClassesFromClass exported_classes  ++
-                  map getMentionedTyConsAndClassesFromTyCon exported_tycons   ++
-                  map getMentionedTyConsAndClassesFromId    exported_vals     ++
-                  map getMentionedTyConsAndClassesFromInstInfo exported_inst_infos)
-
-       mentionable_classes
-         = filter is_mentionable (bagToList mentioned_classes)
-       mentionable_tycons
-         = [ tc | tc <- bagToList mentioned_tycons,
-                  is_mentionable tc,
-                  not (isPrimTyCon tc) ]
-
-       nondup_mentioned_tycons  = fst (removeDups cmp mentionable_tycons)
-       nondup_mentioned_classes = fst (removeDups cmp mentionable_classes)
-
-       -- Next: as discussed in the notes, we want the top-level
-       -- Ids straight from the final STG code, so we can use
-       -- their IdInfos to print pragmas; we slurp them out here,
-       -- then pass them to the printing functions, which may
-       -- use them.
-
-       better_ids = collectExportedStgBinders stg_binds
-
-       -- Make a lookup function for convenient access:
-
-       better_id_fn i
-         = if not (isLocallyDefined i)
-           then i  -- can't be among our "better_ids"
-           else
-              let
-                  eq_fn = if isTopLevId i -- can't trust uniqs
-                          then (\ x y -> getOrigName x == getOrigName y)
-                          else eqId
-              in
-              case [ x | x <- better_ids, x `eq_fn` i ] of
-                []  -> pprPanic "better_id_fn:" (ppr PprShowAll i)
-                       i
-                [x] -> x
-                _   -> panic "better_id_fn"
-
-       -- Finally, we sort everything lexically, so that we always
-       -- get the same interface from the same information:
-
-       sorted_mentioned_tycons  = sortLt ltLexical nondup_mentioned_tycons
-       sorted_mentioned_classes = sortLt ltLexical nondup_mentioned_classes
-
-       sorted_tycons     = sortLt ltLexical exported_tycons
-       sorted_classes    = sortLt ltLexical exported_classes
-       sorted_vals       = sortLt ltLexical exported_vals
-       sorted_inst_infos = sortLt lt_lexical_inst_info exported_inst_infos
+writeIface this_mod old_iface new_iface
+          local_tycons local_classes inst_info
+          final_ids tidy_binds tidy_orphan_rules
+  = 
+    if isNothing opt_HiDir && isNothing opt_HiFile
+       then return ()  -- not producing any .hi file
+       else 
+
+    let 
+       hi_suf = case opt_HiSuf of { Nothing -> "hi"; Just suf -> suf }
+       filename = case opt_HiFile of {
+                       Just f  -> f;
+                       Nothing -> 
+                  case opt_HiDir of {
+                       Just dir -> dir ++ '/':moduleUserString this_mod 
+                                       ++ '.':hi_suf;
+                       Nothing  -> panic "writeIface"
+               }}
     in
-    if (any_purely_local sorted_tycons sorted_classes sorted_vals) then
-       -- this will be less of a HACK when we teach
-       -- mkInterface to do I/O (WDP 94/10)
-       error "Can't produce interface file because of errors!\n"
-    else
-    ppAboves
-       [ppPStr SLIT("{-# GHC_PRAGMA INTERFACE VERSION 7 #-}"),
-       ppCat [ppPStr SLIT("interface"), ppPStr modname, ppPStr SLIT("where")],
-
-       do_import_decls modname
-               sorted_vals sorted_mentioned_classes sorted_mentioned_tycons,
-               -- Mustn't give the data constructors to do_import_decls,
-               -- because they aren't explicitly imported; their tycon is.
-
-       ppAboves (map do_fixity                                 fixity_decls),
-       ppAboves (map (pprIfaceClass better_id_fn inline_env)   sorted_classes),
-       ppAboves (map (do_tycon      tycon_specs)               sorted_tycons),
-       ppAboves (map (do_value      better_id_fn inline_env)   sorted_vals),
-       ppAboves (map (do_instance   better_id_fn inline_env)   sorted_inst_infos),
-
-       ppChar '\n'
-       ]
+
+    do maybe_final_iface <- checkIface old_iface full_new_iface        
+       case maybe_final_iface of {
+         Nothing -> when opt_D_dump_rn_trace $
+                    putStrLn "Interface file unchanged" ;  -- No need to update .hi file
+
+         Just final_iface ->
+
+       do  let mod_vers_unchanged = case old_iface of
+                                     Just iface -> pi_vers iface == pi_vers final_iface
+                                     Nothing -> False
+          when (mod_vers_unchanged && opt_D_dump_rn_trace) $
+               putStrLn "Module version unchanged, but usages differ; hence need new hi file"
+
+          if_hdl <- openFile filename WriteMode
+          printForIface if_hdl (pprIface final_iface)
+          hClose if_hdl
+    }   
   where
-    any_purely_local tycons classes vals
-      =  any bad_tc tycons || any bad_cl classes || any bad_id vals
-      where
-       bad_cl cl
-         = case (maybePurelyLocalClass cl) of
-             Nothing -> False
-             Just xs -> naughty_trace cl xs
-
-       bad_id id
-         = case (maybePurelyLocalType (idType id)) of
-             Nothing -> False
-             Just xs -> naughty_trace id xs
-
-       bad_tc tc
-         = case (maybePurelyLocalTyCon tc) of
-             Nothing -> False
-             Just xs -> if exported_abs then False else naughty_trace tc xs
-         where
-           exported_abs = case (getExportFlag tc) of { ExportAbs -> True; _ -> False }
-
-       naughty_trace x things
-         = pprTrace "Can't export -- `"
-               (ppBesides [ppr PprForUser x, ppStr "' mentions purely local things: ",
-                       ppInterleave pp'SP things])
-               True
+    full_new_iface = completeIface new_iface local_tycons local_classes
+                                            inst_info final_ids tidy_binds
+                                            tidy_orphan_rules
 \end{code}
 
+
 %************************************************************************
-%*                                                                     *
-\subsection[imports-MkIface]{Generating `import' declarations in an interface}
-%*                                                                     *
+%*                                                                     *
+\subsection{Checking if the new interface is up to date
+%*                                                                     *
 %************************************************************************
 
-We gather up lots of (module, name) pairs for which we might print an
-import declaration.  We sort them, for the usual canonicalisation
-reasons.  NB: We {\em assume} the lists passed in don't have duplicates in
-them!  expect).
-
-All rather horribly turgid (WDP).
-
 \begin{code}
-do_import_decls
-       :: FAST_STRING
-       -> [Id] -> [Class] -> [TyCon]
-       -> Pretty
-
-do_import_decls mod_name vals classes tycons
-  = let
-       -- Conjure up (module, name) pairs for all
-       -- the potentially import-decls things:
+checkIface :: Maybe ParsedIface                -- The old interface, read from M.hi
+          -> ParsedIface               -- The new interface; but with all version numbers = 1
+          -> IO (Maybe ParsedIface)    -- Nothing => no change; no need to write new Iface
+                                       -- Just pi => Here is the new interface to write
+                                       --            with correct version numbers
+               -- The I/O part is just so it can print differences
+
+-- NB: the fixities, declarations, rules are all assumed
+-- to be sorted by increasing order of hsDeclName, so that 
+-- we can compare for equality
+
+checkIface Nothing new_iface
+-- No old interface, so definitely write a new one!
+  = return (Just new_iface)
+
+checkIface (Just iface) new_iface
+  | no_output_change && no_usage_change
+  = return Nothing
+
+  | otherwise          -- Add updated version numbers
+  = do { dumpIfSet opt_D_dump_hi_diffs "Interface file changes" pp_diffs ;
+        return (Just final_iface )}
+       
+  where
+    final_iface = new_iface { pi_vers = new_mod_vers,
+                             pi_fixity = (new_fixity_vers, new_fixities),
+                             pi_rules  = (new_rules_vers,  new_rules),
+                             pi_decls  = final_decls }
+
+    no_usage_change = pi_usages iface == pi_usages new_iface
+
+    no_output_change = no_decl_changed && 
+                      new_fixity_vers == fixity_vers && 
+                      new_rules_vers == rules_vers &&
+                      no_export_change
+
+    no_export_change = pi_exports iface == pi_exports new_iface
+
+    new_mod_vers | no_output_change = mod_vers
+                | otherwise        = bumpVersion mod_vers
+
+    mod_vers = pi_vers iface
+
+    (fixity_vers, fixities) = pi_fixity iface
+    (_,       new_fixities) = pi_fixity new_iface
+    new_fixity_vers | fixities == new_fixities = fixity_vers
+                   | otherwise                = bumpVersion fixity_vers
+
+    (rules_vers, rules) = pi_rules iface
+    (_,      new_rules) = pi_rules new_iface
+    new_rules_vers  | rules == new_rules = rules_vers
+                   | otherwise          = bumpVersion rules_vers
+
+    (no_decl_changed, pp_diffs, final_decls) = merge_decls True empty [] (pi_decls iface) (pi_decls new_iface)
+
+       -- Fill in the version number on the new declarations
+       -- by looking at the old declarations.
+       -- Set the flag if anything changes. 
+       -- Assumes that the decls are sorted by hsDeclName
+    merge_decls ok_so_far pp acc []  []        = (ok_so_far, pp, reverse acc)
+    merge_decls ok_so_far pp acc old []        = (False,     pp, reverse acc)
+    merge_decls ok_so_far pp acc [] (nvd:nvds) = merge_decls False (pp $$ only_new nvd) (nvd:acc) [] nvds
+    merge_decls ok_so_far pp acc (vd@(v,d):vds) (nvd@(_,nd):nvds)
+       = case d_name `compare` nd_name of
+               LT -> merge_decls False (pp $$ only_old vd)  acc       vds      (nvd:nvds)
+               GT -> merge_decls False (pp $$ only_new nvd) (nvd:acc) (vd:vds) nvds
+               EQ | d == nd   -> merge_decls ok_so_far pp                   (vd:acc)                  vds nvds
+                  | otherwise -> merge_decls False     (pp $$ changed d nd) ((bumpVersion v, nd):acc) vds nvds
+       where
+         d_name  = hsDeclName d
+         nd_name = hsDeclName nd
+
+    only_old (_,d) = ptext SLIT("Only in old iface:") <+> ppr d
+    only_new (_,d) = ptext SLIT("Only in new iface:") <+> ppr d
+    changed d nd   = ptext SLIT("Changed in iface: ") <+> ((ptext SLIT("Old:") <+> ppr d) $$ 
+                                                          (ptext SLIT("New:") <+> ppr nd))
+\end{code}
 
-       vals_names, classes_names, tycons_names :: [(FAST_STRING, FAST_STRING, [Maybe FAST_STRING])]
-       vals_names      = map get_val_pair   vals
-       classes_names   = map get_class_pair classes
-       tycons_names    = map get_tycon_pair tycons
 
-       -- sort the (module, name) pairs and chop
-       -- them into per-module groups:
 
-       ie_list = sortLt lt (tycons_names ++ classes_names ++ vals_names)
+%************************************************************************
+%*                                                                     *
+\subsection{Printing the interface}
+%*                                                                     *
+%************************************************************************
 
-       per_module_groups = runs same_module ie_list
-    in
-    ppAboves (map print_a_decl per_module_groups)
+\begin{code}
+pprIface (ParsedIface { pi_mod = mod, pi_vers = mod_vers, pi_orphan = orphan,
+                       pi_usages = usages, pi_exports = exports, 
+                       pi_fixity = (fix_vers, fixities),
+                       pi_insts = insts, pi_decls = decls, 
+                       pi_rules = (rule_vers, rules), pi_deprecs = deprecs })
+ = vcat [ ptext SLIT("__interface")
+               <+> doubleQuotes (ptext opt_InPackage)
+               <+> ppr mod <+> ppr mod_vers <+> pp_sub_vers
+               <+> (if orphan then char '!' else empty)
+               <+> int opt_HiVersion
+               <+> ptext SLIT("where")
+       , vcat (map pprExport exports)
+       , vcat (map pprUsage usages)
+       , pprFixities fixities
+       , vcat [ppr i <+> semi | i <- insts]
+       , vcat [ppr_vers v <+> ppr d <> semi | (v,d) <- decls]
+       , pprRules rules
+       , pprDeprecs deprecs
+       ]
   where
-    lt, same_module :: (FAST_STRING, FAST_STRING)
-                   -> (FAST_STRING, FAST_STRING) -> Bool
-
-    lt (m1, ie1, ie2)
-      = case (_CMP_STRING_ m1 m2) of { LT_ -> True; EQ_ -> ie1 < ie2; GT__ -> False }
-
-    same_module (m1, _, _) (m2, _, _) = m1 == m2
-
-    compiling_the_prelude = opt_CompilingPrelude
-
-    print_a_decl :: [(FAST_STRING, FAST_STRING, [Maybe FAST_STRING])] -> Pretty
-    {-
-       Obviously, if the module in question is this one,
-       don't print an import declaration.
-
-       If it's a Prelude* module, we don't print the TyCons/
-       Classes, because the compiler supposedly knows about
-       them already (and they are PreludeCore things anyway).
-
-       But if we are compiling a Prelude module, then we
-       try to do it as "normally" as possible.
-    -}
-    print_a_decl (ielist@((m,_,_) : _))
-      |  m == mod_name
-      || (not compiling_the_prelude &&
-         (m == pRELUDE_CORE || m == pRELUDE_BUILTIN))
-      = ppNil
-
-      | otherwise
-      = ppBesides [ppPStr SLIT("import "), ppPStr m, ppLparen,
-                  ppIntersperse pp'SP{-'-} (map pp_str [n | (_,n,_) <- ielist]),
-                  ppRparen
-                 ]
-      where
-       isnt_tycon_ish :: FAST_STRING -> Bool
-       isnt_tycon_ish str = not (isConop str)
-
-       grab_non_Nothings :: [[Maybe FAST_STRING]] -> [FAST_STRING]
-
-       grab_non_Nothings rns = catMaybes (concat rns)
-
-       pp_str :: FAST_STRING -> Pretty
-       pp_str pstr
-         = if isAvarop pstr then ppStr ("("++str++")") else ppPStr pstr
-         where
-           str = _UNPK_ pstr
+    ppr_vers v | v == initialVersion = empty
+              | otherwise           = int v
+    pp_sub_vers 
+       | fix_vers == initialVersion && rule_vers == initialVersion = empty
+       | otherwise = brackets (ppr fix_vers <+> ppr rule_vers)
 \end{code}
 
-\begin{code}
-get_val_pair   :: Id    -> (FAST_STRING, FAST_STRING)
-get_class_pair :: Class -> (FAST_STRING, FAST_STRING)
-get_tycon_pair :: TyCon -> (FAST_STRING, FAST_STRING)
-
-get_val_pair id
-  = generic_pair id
-
-get_class_pair clas
-  = case (generic_pair clas) of { (orig_mod, orig_nm) ->
-    let
-       nm_to_print = case (getExportFlag clas) of
-                       ExportAll   -> orig_nm _APPEND_ SLIT("(..)") -- nothing like a good HACK!
-                       ExportAbs   -> orig_nm
-                       NotExported -> orig_nm
-    in
-    (orig_mod, nm_to_print) }
+When printing export lists, we print like this:
+       Avail   f               f
+       AvailTC C [C, x, y]     C(x,y)
+       AvailTC C [x, y]        C!(x,y)         -- Exporting x, y but not C
 
-get_tycon_pair tycon
-  = case (generic_pair tycon) of { (orig_mod, orig_nm) ->
-    let
-       nm_to_print = case (getExportFlag tycon) of
-                       ExportAll   -> orig_nm _APPEND_ SLIT("(..)") -- nothing like a good HACK!
-                       ExportAbs   -> orig_nm
-                       NotExported -> orig_nm
-
-       cons        = tyConDataCons tycon
-    in
-    (orig_mod, nm_to_print) }
-
-generic_pair thing
-  = case (getOrigName       thing) of { (orig_mod, orig_nm) ->
-    case (getOccName thing) of { occur_name ->
-    (orig_mod, orig_nm) }}
+\begin{code}
+pprExport :: ExportItem -> SDoc
+pprExport (mod, items)
+ = hsep [ ptext SLIT("__export "), ppr mod, hsep (map upp_avail items) ] <> semi
+  where
+    upp_avail :: RdrAvailInfo -> SDoc
+    upp_avail (Avail name)      = pprOccName name
+    upp_avail (AvailTC name []) = empty
+    upp_avail (AvailTC name ns) = hcat [pprOccName name, bang, upp_export ns']
+                               where
+                                 bang | name `elem` ns = empty
+                                      | otherwise      = char '|'
+                                 ns' = filter (/= name) ns
+    
+    upp_export []    = empty
+    upp_export names = braces (hsep (map pprOccName names))
 \end{code}
 
-%************************************************************************
-%*                                                                     *
-\subsection[fixities-MkIface]{Generating fixity declarations in an interface}
-%*                                                                     *
-%************************************************************************
-
 
 \begin{code}
-do_fixity :: -> RenamedFixityDecl -> Pretty
-
-do_fixity fixity_decl
-  = case (getExportFlag (get_name fixity_decl)) of
-      ExportAll -> ppr PprInterface fixity_decl
-      _                -> ppNil
+pprUsage :: ImportVersion OccName -> SDoc
+pprUsage (m, has_orphans, is_boot, whats_imported)
+  = hsep [ptext SLIT("import"), pprModuleName m, 
+         pp_orphan, pp_boot,
+         upp_import_versions whats_imported
+    ] <> semi
   where
-     get_name (InfixL n _) = n
-     get_name (InfixR n _) = n
-     get_name (InfixN n _) = n
+    pp_orphan | has_orphans = char '!'
+             | otherwise   = empty
+    pp_boot   | is_boot     = char '@'
+              | otherwise   = empty
+
+       -- Importing the whole module is indicated by an empty list
+    upp_import_versions NothingAtAll   = empty
+    upp_import_versions (Everything v) = dcolon <+> int v
+    upp_import_versions (Specifically vm vf vr nvs)
+      = dcolon <+> int vm <+> int vf <+> int vr <+> hsep [ ppr n <+> int v | (n,v) <- nvs ]
 \end{code}
 
-%************************************************************************
-%*                                                                     *
-\subsection[tycons-MkIface]{Generating tycon declarations in an interface}
-%*                                                                     *
-%************************************************************************
 
 \begin{code}
-do_tycon :: FiniteMap TyCon [(Bool, [Maybe Type])] -> TyCon -> Pretty
+pprFixities []    = empty
+pprFixities fixes = hsep (map ppr fixes) <> semi
 
-do_tycon tycon_specs_map tycon
-  = pprTyCon PprInterface tycon tycon_specs
-  where
-    tycon_specs = map snd (lookupWithDefaultFM tycon_specs_map [] tycon)
+pprRules []    = empty
+pprRules rules = hsep [ptext SLIT("{-## __R"), hsep (map ppr rules), ptext SLIT("##-}")]
+
+pprDeprecs []   = empty
+pprDeprecs deps = hsep [ ptext SLIT("{-## __D"), guts, ptext SLIT("##-}")]
+               where
+                 guts = hsep [ ppr ie <+> doubleQuotes (ppr txt) <> semi 
+                             | Deprecation ie txt _ <- deps ]
 \end{code}
 
+
 %************************************************************************
-%*                                                                     *
-\subsection[values-MkIface]{Generating a value's signature in an interface}
-%*                                                                     *
+%*                                                                     *
+\subsection{Completing the new interface}
+%*                                                                     *
 %************************************************************************
 
 \begin{code}
-do_value :: (Id -> Id)
-        -> IdEnv UnfoldingDetails
-        -> Id
-        -> Pretty
-
-do_value better_id_fn inline_env val
-  = let
-       sty         = PprInterface
-       better_val  = better_id_fn val
-       name_str    = getOccName better_val -- NB: not orig name!
-
-       id_info     = getIdInfo better_val
-
-       val_ty      = let
-                        orig_ty  = idType val
-                        final_ty = idType better_val
-                     in
---                   ASSERT (orig_ty == final_ty || mkLiftTy orig_ty == final_ty)
-                     ASSERT (if (orig_ty == final_ty || mkLiftTy orig_ty == final_ty) then True else pprTrace "do_value:" (ppCat [ppr PprDebug val, ppr PprDebug better_val]) False)
-                     orig_ty
-
-       -- Note: We export the type of the original val
-       -- The type of an unboxed val will have been *lifted* by the desugarer
-       -- In this case we export an unlifted type, but id_info which assumes
-       --   a lifted Id i.e. extracted from better_val (above)
-       -- The importing module must lift the Id before using the imported id_info
-
-       pp_id_info
-         = if opt_OmitInterfacePragmas
-           || boringIdInfo id_info
-           then ppNil
-           else ppCat [ppPStr SLIT("\t{-# GHC_PRAGMA"),
-                       ppIdInfo sty better_val True{-yes specs-}
-                           better_id_fn inline_env id_info,
-                       ppPStr SLIT("#-}")]
-    in
-    ppAbove (ppCat [ppr_non_op name_str,
-                   ppPStr SLIT("::"), pprGenType sty val_ty])
-           pp_id_info
+completeIface new_iface local_tycons local_classes
+                       inst_info final_ids tidy_binds
+                       tidy_orphan_rules
+  = new_iface { pi_decls = [(initialVersion,d) | d <- sortLt lt_decl all_decls],
+               pi_insts = sortLt lt_inst_decl inst_dcls,
+               pi_rules = (initialVersion, rule_dcls)
+    }
+  where
+     all_decls = cls_dcls ++ ty_dcls ++ bagToList val_dcls
+     (inst_dcls, inst_ids) = ifaceInstances inst_info
+     cls_dcls = map ifaceClass local_classes
+  
+     ty_dcls  = map ifaceTyCon (filter (not . isWiredInName . getName) local_tycons)
+
+     (val_dcls, emitted_ids) = ifaceBinds (inst_ids `unionVarSet` orphan_rule_ids)
+                                         final_ids tidy_binds
 
--- sadly duplicates Outputable.pprNonOp (ToDo)
+     rule_dcls | opt_OmitInterfacePragmas = []
+              | otherwise                = ifaceRules tidy_orphan_rules emitted_ids
 
-ppr_non_op str
-  = if isAvarop str -- NOT NEEDED: || isAconop
-    then ppBesides [ppLparen, ppPStr str, ppRparen]
-    else ppPStr str
+     orphan_rule_ids = unionVarSets [ ruleSomeFreeVars interestingId rule 
+                                   | ProtoCoreRule _ _ rule <- tidy_orphan_rules]
+
+lt_decl      d1 d2 = hsDeclName   d1 < hsDeclName d2
+lt_inst_decl d1 d2 = instDeclName d1 < instDeclName d2
+       -- Even instance decls have names, namely the dfun name
 \end{code}
 
+
 %************************************************************************
-%*                                                                     *
-\subsection[instances-MkIface]{Generating instance declarations in an interface}
-%*                                                                     *
+%*                                                                     *
+\subsection{Completion stuff}
+%*                                                                     *
 %************************************************************************
 
-The types of ``dictionary functions'' (dfuns) have just the required
-info for instance declarations in interfaces.  However, the dfuns that
-GHC really uses have {\em extra} dictionaries passed to them (for
-efficiency).  When we print interfaces, we want to omit that
-dictionary information.  (It can be reconsituted on the other end,
-from instance and class decls).
+\begin{code}
+ifaceRules :: [ProtoCoreRule] -> IdSet -> [RdrNameRuleDecl]
+ifaceRules rules emitted
+  = orphan_rules ++ local_rules
+  where
+    orphan_rules = [ toHsRule fn rule | ProtoCoreRule _ fn rule <- rules ]
+    local_rules  = [ toHsRule fn rule
+                  | fn <- varSetElems emitted, 
+                    rule <- rulesRules (idSpecialisation fn),
+                    not (isBuiltinRule rule),
+                               -- We can't print builtin rules in interface files
+                               -- Since they are built in, an importing module
+                               -- will have access to them anyway
+
+                       -- Sept 00: I've disabled this test.  It doesn't stop many, if any, rules
+                       -- from coming out, and to make it work properly we need to add 
+                            all (`elemVarSet` emitted) (varSetElems (ruleSomeLhsFreeVars interestingId rule))
+                               -- Spit out a rule only if all its lhs free vars are emitted
+                               -- This is a good reason not to do it when we emit the Id itself
+                  ]
+\end{code}
+
+\begin{code}                    
+ifaceInstances :: Bag InstInfo -> ([RdrNameInstDecl], IdSet)
+                  -- The IdSet is the needed dfuns
+
+ifaceInstances inst_infos
+  = (decls, needed_ids)
+  where                        
+    decls       = map to_decl togo_insts
+    togo_insts = filter is_togo_inst (bagToList inst_infos)
+    needed_ids  = mkVarSet [dfun_id | InstInfo _ _ _ _ dfun_id _ _ _ <- togo_insts]
+    is_togo_inst (InstInfo _ _ _ _ dfun_id _ _ _) = isLocallyDefined dfun_id
+                                
+    -------                     
+    to_decl (InstInfo clas tvs tys theta dfun_id _ _ _)
+      = let                     
+               -- The deNoteType is very important.   It removes all type
+               -- synonyms from the instance type in interface files.
+               -- That in turn makes sure that when reading in instance decls
+               -- from interface files that the 'gating' mechanism works properly.
+               -- Otherwise you could have
+               --      type Tibble = T Int
+               --      instance Foo Tibble where ...
+               -- and this instance decl wouldn't get imported into a module
+               -- that mentioned T but not Tibble.
+           forall_ty     = mkSigmaTy tvs theta (deNoteType (mkDictTy clas tys))
+           tidy_ty = tidyTopType forall_ty
+       in                       
+       InstDecl (toHsType tidy_ty) EmptyMonoBinds [] (Just (toRdrName dfun_id)) noSrcLoc 
+\end{code}
 
 \begin{code}
-do_instance :: (Id -> Id)
-           -> IdEnv UnfoldingDetails
-           -> InstInfo
-           -> Pretty
-
-do_instance better_id_fn inline_env
-    (InstInfo clas tv_tmpls ty inst_decl_theta dfun_theta dfun_id constm_ids _ from_here modname _ _)
-  = let
-       sty = PprInterface
-
-       better_dfun      = better_id_fn dfun_id
-       better_dfun_info = getIdInfo better_dfun
-       better_constms   = map better_id_fn constm_ids
-
-       class_op_strs = map getClassOpString (getClassOps clas)
-
-       pragma_begin
-         = ppCat [ppPStr SLIT("\t{-# GHC_PRAGMA"), pp_modname, ppPStr SLIT("{-dfun-}"),
-                  ppIdInfo sty better_dfun False{-NO specs-}
-                   better_id_fn inline_env better_dfun_info]
-
-       pragma_end = ppPStr SLIT("#-}")
-
-       pp_modname = if _NULL_ modname
-                    then ppNil
-                    else ppCat [ppStr "_M_", ppPStr modname]
-
-       name_pragma_pairs
-         = pp_the_list [ ppCat [ppChar '\t', ppr_non_op op, ppEquals,
-                                ppChar '{' ,
-                                ppIdInfo sty constm True{-YES, specs-}
-                                 better_id_fn inline_env
-                                 (getIdInfo constm),
-                                ppChar '}' ]
-                       | (op, constm) <- class_op_strs `zip` better_constms ]
-
-#ifdef DEBUG
-       pp_the_list [] = panic "MkIface: no class_ops or better_constms?"
-#endif
-       pp_the_list [p]    = p
-       pp_the_list (p:ps) = ppAbove (ppBeside p ppComma) (pp_the_list ps)
-
-       real_stuff
-         = ppCat [ppPStr SLIT("instance"),
-                  ppr sty (mkSigmaTy tv_tmpls inst_decl_theta (mkDictTy clas ty))]
-    in
-    if opt_OmitInterfacePragmas
-    || boringIdInfo better_dfun_info
-    then real_stuff
-    else ppAbove real_stuff
-         ({-ppNest 8 -} -- ppNest does nothing
-            if null better_constms
-            then ppCat [pragma_begin, pragma_end]
-            else ppAbove pragma_begin (ppCat [name_pragma_pairs, pragma_end])
-         )
+ifaceTyCon :: TyCon -> RdrNameHsDecl
+ifaceTyCon tycon
+  | isSynTyCon tycon
+  = TyClD (TySynonym (toRdrName tycon)
+                    (toHsTyVars tyvars) (toHsType ty)
+                    noSrcLoc)
+  where
+    (tyvars, ty) = getSynTyConDefn tycon
+
+ifaceTyCon tycon
+  | isAlgTyCon tycon
+  = TyClD (mkTyData new_or_data (toHsContext (tyConTheta tycon))
+                 (toRdrName tycon)
+                 (toHsTyVars tyvars)
+                 (map ifaceConDecl (tyConDataCons tycon))
+                 (tyConFamilySize tycon)
+                 Nothing NoDataPragmas noSrcLoc)
+  where
+    tyvars = tyConTyVars tycon
+    new_or_data | isNewTyCon tycon = NewType
+               | otherwise        = DataType
+
+    ifaceConDecl data_con 
+       = ConDecl (toRdrName data_con) (error "ifaceConDecl")
+                 (toHsTyVars ex_tyvars)
+                 (toHsContext ex_theta)
+                 details noSrcLoc
+       where
+         (tyvars1, _, ex_tyvars, ex_theta, arg_tys, tycon1) = dataConSig data_con
+          field_labels   = dataConFieldLabels data_con
+          strict_marks   = dataConStrictMarks data_con
+         details
+           | null field_labels
+           = ASSERT( tycon == tycon1 && tyvars == tyvars1 )
+             VanillaCon (zipWith mk_bang_ty strict_marks arg_tys)
+
+           | otherwise
+           = RecCon (zipWith mk_field strict_marks field_labels)
+
+    mk_bang_ty NotMarkedStrict     ty = Unbanged (toHsType ty)
+    mk_bang_ty (MarkedUnboxed _ _) ty = Unpacked (toHsType ty)
+    mk_bang_ty MarkedStrict        ty = Banged   (toHsType ty)
+
+    mk_field strict_mark field_label
+       = ([toRdrName field_label], mk_bang_ty strict_mark (fieldLabelType field_label))
+
+ifaceTyCon tycon
+  = pprPanic "pprIfaceTyDecl" (ppr tycon)
+
+ifaceClass clas
+  = TyClD (ClassDecl (toHsContext sc_theta)
+                    (toRdrName clas)
+                    (toHsTyVars clas_tyvars)
+                    (toHsFDs clas_fds)
+                    (map toClassOpSig op_stuff)
+                    EmptyMonoBinds NoClassPragmas
+                    [] noSrcLoc
+    )
+  where
+     bogus = error "ifaceClass"
+     (clas_tyvars, clas_fds, sc_theta, _, op_stuff) = classExtraBigSig clas
+
+     toClassOpSig (sel_id, def_meth) = 
+       ASSERT(sel_tyvars == clas_tyvars)
+         ClassOpSig (toRdrName sel_id) (Just def_meth') (toHsType op_ty) noSrcLoc
+       where
+         (sel_tyvars, _, op_ty) = splitSigmaTy (idType sel_id)
+         def_meth' = case def_meth of
+                        NoDefMeth  -> NoDefMeth
+                        GenDefMeth -> GenDefMeth
+                        DefMeth id -> DefMeth (toRdrName id)
 \end{code}
 
+
 %************************************************************************
-%*                                                                     *
-\subsection[utils-InstInfos]{Utility functions for @InstInfos@}
-%*                                                                     *
+%*                                                                     *
+\subsection{Value bindings}
+%*                                                                     * 
 %************************************************************************
 
-ToDo: perhaps move.
-
-Classes/TyCons are ``known,'' more-or-less.  Prelude TyCons are
-``completely'' known---they don't need to be mentioned in interfaces.
-Classes usually don't need to be mentioned in interfaces, but if we're
-compiling the prelude, then we treat them without special favours.
 \begin{code}
-is_exportable_tycon_or_class export_list_fns tc
-  = if not (fromPreludeCore tc) then
-       True
-    else
-       in_export_list_or_among_dotdot_modules
-           opt_CompilingPrelude -- ignore M.. stuff if compiling prelude
-           export_list_fns tc
-
-in_export_list_or_among_dotdot_modules ignore_Mdotdots (in_export_list, among_dotdot_modules) tc
-  = if in_export_list (getOccName tc) then
-       True
-    else
---     pprTrace "in_export:" (ppAbove (ppr PprDebug ignore_Mdotdots) (ppPStr (getOccName  tc))) (
-    if ignore_Mdotdots then
-       False
-    else
-       any among_dotdot_modules (getInformingModules tc)
---  )
-
-is_mentionable tc
-  = not (from_PreludeCore_or_Builtin tc) || opt_CompilingPrelude
+ifaceBinds :: IdSet            -- These Ids are needed already
+          -> [Id]              -- Ids used at code-gen time; they have better pragma info!
+          -> [CoreBind]        -- In dependency order, later depend on earlier
+          -> (Bag RdrNameHsDecl, IdSet)                -- Set of Ids actually spat out
+
+ifaceBinds needed_ids final_ids binds
+  = go needed_ids (reverse binds) emptyBag emptyVarSet 
+               -- Reverse so that later things will 
+               -- provoke earlier ones to be emitted
   where
-    from_PreludeCore_or_Builtin thing
-      = let
-           mod_name = fst (getOrigName thing)
-       in
-       mod_name == pRELUDE_CORE || mod_name == pRELUDE_BUILTIN
-
-is_exported_inst_info export_list_fns
-       (InstInfo clas _ ty _ _ _ _ _ from_here _ _ _)
-  = let
-       seems_exported = instanceIsExported clas ty from_here
-       (tycon, _, _) = getAppTyCon ty
-    in
-    if (opt_OmitReexportedInstances && not from_here) then
-       False -- Flag says to violate Haskell rules, blatantly
-
-    else if not opt_CompilingPrelude
-        || not (isFunTyCon tycon || fromPreludeCore tycon)
-        || not (fromPreludeCore clas) then
-       seems_exported -- take what we got
-
-    else -- compiling Prelude & tycon/class are Prelude things...
-       from_here
-       || in_export_list_or_among_dotdot_modules True{-ignore M..s-} export_list_fns clas
-       || in_export_list_or_among_dotdot_modules True{-ignore M..s-} export_list_fns tycon
+    final_id_map  = listToUFM [(id,id) | id <- final_ids]
+    get_idinfo id = case lookupUFM final_id_map id of
+                       Just id' -> idInfo id'
+                       Nothing  -> pprTrace "ifaceBinds not found:" (ppr id) $
+                                   idInfo id
+
+       -- The 'needed' set contains the Ids that are needed by earlier
+       -- interface file emissions.  If the Id isn't in this set, and isn't
+       -- exported, there's no need to emit anything
+    need_id needed_set id = id `elemVarSet` needed_set || isUserExportedId id 
+
+    go needed [] decls emitted
+       | not (isEmptyVarSet needed) = pprTrace "ifaceBinds: free vars:" 
+                                         (sep (map ppr (varSetElems needed)))
+                                      (decls, emitted)
+       | otherwise                  = (decls, emitted)
+
+    go needed (NonRec id rhs : binds) decls emitted
+       | need_id needed id
+       = if omitIfaceSigForId id then
+           go (needed `delVarSet` id) binds decls (emitted `extendVarSet` id)
+         else
+           go ((needed `unionVarSet` extras) `delVarSet` id)
+              binds
+              (decl `consBag` decls)
+              (emitted `extendVarSet` id)
+       | otherwise
+       = go needed binds decls emitted
+       where
+         (decl, extras) = ifaceId get_idinfo False id rhs
+
+       -- Recursive groups are a bit more of a pain.  We may only need one to
+       -- start with, but it may call out the next one, and so on.  So we
+       -- have to look for a fixed point.  We don't want necessarily them all, 
+       -- because without -O we may only need the first one (if we don't emit
+       -- its unfolding)
+    go needed (Rec pairs : binds) decls emitted
+       = go needed' binds decls' emitted' 
+       where
+         (new_decls, new_emitted, extras) = go_rec needed pairs
+         decls'   = new_decls `unionBags` decls
+         needed'  = (needed `unionVarSet` extras) `minusVarSet` mkVarSet (map fst pairs) 
+         emitted' = emitted `unionVarSet` new_emitted
+
+    go_rec :: IdSet -> [(Id,CoreExpr)] -> (Bag RdrNameHsDecl, IdSet, IdSet)
+    go_rec needed pairs
+       | null decls = (emptyBag, emptyVarSet, emptyVarSet)
+       | otherwise  = (more_decls   `unionBags`   listToBag decls, 
+                       more_emitted `unionVarSet` mkVarSet (map fst needed_prs),
+                       more_extras  `unionVarSet` extras)
+       where
+         (needed_prs,leftover_prs) = partition is_needed pairs
+         (decls, extras_s)         = unzip [ifaceId get_idinfo True id rhs 
+                                           | (id,rhs) <- needed_prs, not (omitIfaceSigForId id)]
+         extras                    = unionVarSets extras_s
+         (more_decls, more_emitted, more_extras) = go_rec extras leftover_prs
+         is_needed (id,_) = need_id needed id
 \end{code}
 
-\begin{code}
-lt_lexical_inst_info (InstInfo _ _ _ _ _ dfun1 _ _ _ _ _ _) (InstInfo _ _ _ _ _ dfun2 _ _ _ _ _ _)
-  = ltLexical dfun1 dfun2
-\end{code}
 
 \begin{code}
-getMentionedTyConsAndClassesFromInstInfo (InstInfo clas _ ty _ dfun_theta _ _ _ _ _ _ _)
-  = case (getMentionedTyConsAndClassesFromType ty) of { (ts, cs) ->
-    case [ c | (c, _) <- dfun_theta ]                        of { theta_classes ->
-    (ts, (cs `unionBags` listToBag theta_classes) `snocBag` clas)
-    }}
+ifaceId :: (Id -> IdInfo)      -- This function "knows" the extra info added
+                               -- by the STG passes.  Sigh
+       -> Bool                 -- True <=> recursive, so don't print unfolding
+       -> Id
+       -> CoreExpr             -- The Id's right hand side
+       -> (RdrNameHsDecl, IdSet)       -- The emitted stuff, plus any *extra* needed Ids
+
+ifaceId get_idinfo is_rec id rhs
+  = (SigD (IfaceSig (toRdrName id) (toHsType id_type) hs_idinfo noSrcLoc),  new_needed_ids)
+  where
+    id_type     = idType id
+    core_idinfo = idInfo id
+    stg_idinfo  = get_idinfo id
+
+    hs_idinfo | opt_OmitInterfacePragmas = []
+             | otherwise                = arity_hsinfo  ++ caf_hsinfo  ++ cpr_hsinfo ++ 
+                                          strict_hsinfo ++ wrkr_hsinfo ++ unfold_hsinfo
+
+    ------------  Arity  --------------
+    arity_info   = arityInfo stg_idinfo
+    stg_arity   = arityLowerBound arity_info
+    arity_hsinfo = case arityInfo stg_idinfo of
+                       a@(ArityExactly n) -> [HsArity a]
+                       other              -> []
+
+    ------------ Caf Info --------------
+    caf_hsinfo = case cafInfo stg_idinfo of
+                  NoCafRefs -> [HsNoCafRefs]
+                  otherwise -> []
+
+    ------------ CPR Info --------------
+    cpr_hsinfo = case cprInfo core_idinfo of
+                  ReturnsCPR -> [HsCprInfo]
+                  NoCPRInfo  -> []
+
+    ------------  Strictness  --------------
+    strict_info   = strictnessInfo core_idinfo
+    bottoming_fn  = isBottomingStrictness strict_info
+    strict_hsinfo = case strict_info of
+                       NoStrictnessInfo -> []
+                       info             -> [HsStrictness info]
+
+
+    ------------  Worker  --------------
+       -- We only treat a function as having a worker if
+       -- the exported arity (which is now the number of visible lambdas)
+       -- is the same as the arity at the moment of the w/w split
+       -- If so, we can safely omit the unfolding inside the wrapper, and
+       -- instead re-generate it from the type/arity/strictness info
+       -- But if the arity has changed, we just take the simple path and
+       -- put the unfolding into the interface file, forgetting the fact
+       -- that it's a wrapper.  
+       --
+       -- How can this happen?  Sometimes we get
+       --      f = coerce t (\x y -> $wf x y)
+       -- at the moment of w/w split; but the eta reducer turns it into
+       --      f = coerce t $wf
+       -- which is perfectly fine except that the exposed arity so far as
+       -- the code generator is concerned (zero) differs from the arity
+       -- when we did the split (2).  
+       --
+       -- All this arises because we use 'arity' to mean "exactly how many
+       -- top level lambdas are there" in interface files; but during the
+       -- compilation of this module it means "how many things can I apply
+       -- this to".
+    work_info           = workerInfo core_idinfo
+    HasWorker work_id _ = work_info
+
+    has_worker = case work_info of
+                 HasWorker work_id wrap_arity 
+                  | wrap_arity == stg_arity -> True
+                  | otherwise               -> pprTrace "ifaceId: arity change:" (ppr id) 
+                                               False
+                                                         
+                 other                      -> False
+
+    wrkr_hsinfo | has_worker = [HsWorker (toRdrName work_id)]
+               | otherwise  = []
+
+    ------------  Unfolding  --------------
+    inline_pragma  = inlinePragInfo core_idinfo
+    dont_inline           = isNeverInlinePrag inline_pragma
+
+    unfold_hsinfo | show_unfold = [HsUnfold inline_pragma (toUfExpr rhs)]
+                 | otherwise   = []
+
+    show_unfold = not has_worker        &&     -- Not unnecessary
+                 not bottoming_fn       &&     -- Not necessary
+                 not dont_inline        &&
+                 not loop_breaker       &&
+                 rhs_is_small           &&     -- Small enough
+                 okToUnfoldInHiFile rhs        -- No casms etc
+
+    rhs_is_small = couldBeSmallEnoughToInline opt_UF_HiFileThreshold rhs
+
+    ------------  Specialisations --------------
+    spec_info   = specInfo core_idinfo
+    
+    ------------  Occ info  --------------
+    loop_breaker  = isLoopBreaker (occInfo core_idinfo)
+
+    ------------  Extra free Ids  --------------
+    new_needed_ids | opt_OmitInterfacePragmas = emptyVarSet
+                  | otherwise                = worker_ids      `unionVarSet`
+                                               unfold_ids      `unionVarSet`
+                                               spec_ids
+
+    worker_ids | has_worker && interestingId work_id = unitVarSet work_id
+                       -- Conceivably, the worker might come from
+                       -- another module
+              | otherwise = emptyVarSet
+
+    spec_ids = filterVarSet interestingId (rulesRhsFreeVars spec_info)
+
+    unfold_ids | show_unfold = find_fvs rhs
+              | otherwise   = emptyVarSet
+
+    find_fvs expr = exprSomeFreeVars interestingId expr
+
+interestingId id = isId id && isLocallyDefined id && not (hasNoBinding id)
 \end{code}
+