X-Git-Url: http://git.megacz.com/?a=blobdiff_plain;ds=sidebyside;f=ghc%2Fcompiler%2Frename%2FRnSource.lhs;h=d90f63ad5650c9f7323f17bc07573c298b3d483d;hb=d72417969c6112d5d81669757a4863782e7998fb;hp=40be2b7287d7ea0f0df0be83efa2db76a311482d;hpb=cfbedcecad9f9c4241fa1313e73468fd95db76b3;p=ghc-hetmet.git diff --git a/ghc/compiler/rename/RnSource.lhs b/ghc/compiler/rename/RnSource.lhs index 40be2b7..d90f63a 100644 --- a/ghc/compiler/rename/RnSource.lhs +++ b/ghc/compiler/rename/RnSource.lhs @@ -4,61 +4,64 @@ \section[RnSource]{Main pass of renamer} \begin{code} -module RnSource ( rnDecl, rnSourceDecls, rnHsType, rnHsSigType, rnHsPolyType ) where +module RnSource ( + rnSrcDecls, rnExtCoreDecls, checkModDeprec, + rnTyClDecl, rnIfaceRuleDecl, rnInstDecl, + rnBinds, rnStats, + ) where #include "HsVersions.h" import RnExpr import HsSyn -import HsPragmas -import HsTypes ( getTyVarName, pprHsPred, cmpHsTypes ) -import RdrName ( RdrName, isRdrDataCon, rdrNameOcc, isRdrTyVar ) -import RdrHsSyn ( RdrNameContext, RdrNameHsType, RdrNameConDecl, - extractRuleBndrsTyVars, extractHsTyRdrTyVars, extractHsTysRdrTyVars +import RdrName ( RdrName, isRdrDataCon, elemRdrEnv ) +import RdrHsSyn ( RdrNameConDecl, RdrNameTyClDecl, RdrNameHsDecl, + RdrNameDeprecation, RdrNameFixitySig, + RdrNameHsBinds, + extractGenericPatTyVars ) import RnHsSyn import HsCore -import RnBinds ( rnTopBinds, rnMethodBinds, renameSigs, unknownSigErr ) -import RnEnv ( bindTyVarsRn, lookupBndrRn, lookupOccRn, getIPName, - lookupImplicitOccRn, - bindLocalsRn, bindLocalRn, bindLocalsFVRn, bindUVarRn, - bindTyVarsFVRn, bindTyVarsFV2Rn, extendTyVarEnvFVRn, - bindCoreLocalFVRn, bindCoreLocalsFVRn, - checkDupOrQualNames, checkDupNames, - mkImportedGlobalName, mkImportedGlobalFromRdrName, - newDFunName, getDFunKey, newImplicitBinder, - FreeVars, emptyFVs, plusFV, plusFVs, unitFV, addOneFV, mapFvRn - ) -import RnMonad - -import FunDeps ( oclose ) - -import Name ( Name, OccName, - ExportFlag(..), Provenance(..), - nameOccName, NamedThing(..) +import RnNames ( importsFromLocalDecls ) +import RnTypes ( rnHsType, rnHsSigType, rnHsTypeFVs, rnContext ) + +import RnBinds ( rnTopMonoBinds, rnMonoBinds, rnMethodBinds, + renameSigs, renameSigsFVs ) +import RnEnv ( lookupTopBndrRn, lookupOccRn, lookupSysBndr, + newLocalsRn, lookupGlobalOccRn, + bindLocalsFVRn, bindPatSigTyVars, + bindTyVarsRn, extendTyVarEnvFVRn, + bindCoreLocalRn, bindCoreLocalsRn, bindLocalNames, + checkDupOrQualNames, checkDupNames, mapFvRn, + lookupTopSrcBndr_maybe, lookupTopSrcBndr, + dataTcOccs, unknownNameErr, + plusGlobalRdrEnv ) +import TcRnMonad + +import BasicTypes ( FixitySig(..) ) +import HscTypes ( ExternalPackageState(..), FixityEnv, + Deprecations(..), plusDeprecs ) +import Module ( moduleEnvElts ) +import Class ( FunDep, DefMeth (..) ) +import TyCon ( DataConDetails(..), visibleDataCons ) +import Name ( Name ) import NameSet -import OccName ( mkDefaultMethodOcc ) -import BasicTypes ( TopLevelFlag(..) ) -import FiniteMap ( elemFM ) -import PrelInfo ( derivableClassKeys, - deRefStablePtr_NAME, makeStablePtr_NAME, bindIO_NAME, returnIO_NAME - ) +import NameEnv +import ErrUtils ( dumpIfSet ) +import PrelNames ( newStablePtrName, bindIOName, returnIOName ) +import List ( partition ) import Bag ( bagToList ) -import List ( partition, nub ) import Outputable import SrcLoc ( SrcLoc ) -import CmdLineOpts ( opt_GlasgowExts, opt_WarnUnusedMatches ) -- Warn of unused for-all'd tyvars -import Unique ( Uniquable(..) ) -import UniqFM ( lookupUFM ) -import ErrUtils ( Message ) -import CStrings ( isCLabelString ) -import Maybes ( maybeToBool, catMaybes ) -import Util +import CmdLineOpts ( DynFlag(..) ) + -- Warn of unused for-all'd tyvars +import Maybes ( maybeToBool, seqMaybe ) +import Maybe ( maybe, catMaybes, isNothing ) \end{code} -@rnDecl@ `renames' declarations. +@rnSourceDecl@ `renames' declarations. It simultaneously performs dependency analysis and precedence parsing. It also does the following error checks: \begin{enumerate} @@ -74,200 +77,233 @@ Checks the @(..)@ etc constraints in the export list. \end{enumerate} -%********************************************************* -%* * -\subsection{Value declarations} -%* * -%********************************************************* - \begin{code} -rnSourceDecls :: [RdrNameHsDecl] -> RnMS ([RenamedHsDecl], FreeVars) - -- The decls get reversed, but that's ok - -rnSourceDecls decls +rnSrcDecls :: [RdrNameHsDecl] -> RnM (TcGblEnv, [RenamedHsDecl], FreeVars) + +rnSrcDecls decls + = do { (rdr_env, imports) <- importsFromLocalDecls decls ; + updGblEnv (\gbl -> gbl { tcg_rdr_env = rdr_env `plusGlobalRdrEnv` + tcg_rdr_env gbl, + tcg_imports = imports `plusImportAvails` + tcg_imports gbl }) + $ do { + + -- Deal with deprecations (returns only the extra deprecations) + deprecs <- rnSrcDeprecDecls [d | DeprecD d <- decls] ; + updGblEnv (\gbl -> gbl { tcg_deprecs = tcg_deprecs gbl `plusDeprecs` deprecs }) + $ do { + + -- Deal with top-level fixity decls + -- (returns the total new fixity env) + fix_env <- rnSrcFixityDecls decls ; + updGblEnv (\gbl -> gbl { tcg_fix_env = fix_env }) + $ do { + + -- Rename remaining declarations + (rn_src_decls, src_fvs) <- rn_src_decls decls ; + + tcg_env <- getGblEnv ; + return (tcg_env, rn_src_decls, src_fvs) + }}}} + +rnExtCoreDecls :: [RdrNameHsDecl] -> RnM ([RenamedHsDecl], FreeVars) +rnExtCoreDecls decls = rn_src_decls decls + +rn_src_decls decls -- Declarartions get reversed, but no matter = go emptyFVs [] decls where - -- Fixity decls have been dealt with already; ignore them - go fvs ds' [] = returnRn (ds', fvs) - go fvs ds' (FixD _:ds) = go fvs ds' ds - go fvs ds' (d:ds) = rnDecl d `thenRn` \(d', fvs') -> - go (fvs `plusFV` fvs') (d':ds') ds + -- Fixity and deprecations have been dealt with already; ignore them + go fvs ds' [] = returnM (ds', fvs) + go fvs ds' (FixD _:ds) = go fvs ds' ds + go fvs ds' (DeprecD _:ds) = go fvs ds' ds + go fvs ds' (d:ds) = rnSrcDecl d `thenM` \(d', fvs') -> + go (fvs `plusFV` fvs') (d':ds') ds \end{code} %********************************************************* -%* * -\subsection{Value declarations} -%* * +%* * + Source-code fixity declarations +%* * %********************************************************* \begin{code} --- rnDecl does all the work -rnDecl :: RdrNameHsDecl -> RnMS (RenamedHsDecl, FreeVars) +rnSrcFixityDecls :: [RdrNameHsDecl] -> TcRn m FixityEnv +rnSrcFixityDecls decls + = getGblEnv `thenM` \ gbl_env -> + foldlM rnFixityDecl (tcg_fix_env gbl_env) + fix_decls `thenM` \ fix_env -> + traceRn (text "fixity env" <+> ppr fix_env) `thenM_` + returnM fix_env + where + fix_decls = foldr get_fix_sigs [] decls + + -- Get fixities from top level decls, and from class decl sigs too + get_fix_sigs (FixD fix) acc = fix:acc + get_fix_sigs (TyClD (ClassDecl { tcdSigs = sigs})) acc + = [sig | FixSig sig <- sigs] ++ acc + get_fix_sigs other_decl acc = acc + +rnFixityDecl :: FixityEnv -> RdrNameFixitySig -> TcRn m FixityEnv +rnFixityDecl fix_env (FixitySig rdr_name fixity loc) + = -- GHC extension: look up both the tycon and data con + -- for con-like things + -- If neither are in scope, report an error; otherwise + -- add both to the fixity env + mappM lookupTopSrcBndr_maybe (dataTcOccs rdr_name) `thenM` \ maybe_ns -> + case catMaybes maybe_ns of + [] -> addSrcLoc loc $ + addErr (unknownNameErr rdr_name) `thenM_` + returnM fix_env + ns -> foldlM add fix_env ns + where + add fix_env name + = case lookupNameEnv fix_env name of + Just (FixitySig _ _ loc') -> addErr (dupFixityDecl rdr_name loc loc') `thenM_` + returnM fix_env + Nothing -> returnM (extendNameEnv fix_env name (FixitySig name fixity loc)) + +dupFixityDecl rdr_name loc1 loc2 + = vcat [ptext SLIT("Multiple fixity declarations for") <+> quotes (ppr rdr_name), + ptext SLIT("at ") <+> ppr loc1, + ptext SLIT("and") <+> ppr loc2] +\end{code} -rnDecl (ValD binds) = rnTopBinds binds `thenRn` \ (new_binds, fvs) -> - returnRn (ValD new_binds, fvs) +%********************************************************* +%* * + Source-code deprecations declarations +%* * +%********************************************************* -rnDecl (SigD (IfaceSig name ty id_infos loc)) - = pushSrcLocRn loc $ - lookupBndrRn name `thenRn` \ name' -> - rnHsPolyType doc_str ty `thenRn` \ (ty',fvs1) -> - mapFvRn rnIdInfo id_infos `thenRn` \ (id_infos', fvs2) -> - returnRn (SigD (IfaceSig name' ty' id_infos' loc), fvs1 `plusFV` fvs2) - where - doc_str = text "the interface signature for" <+> quotes (ppr name) +For deprecations, all we do is check that the names are in scope. +It's only imported deprecations, dealt with in RnIfaces, that we +gather them together. + +\begin{code} +rnSrcDeprecDecls :: [RdrNameDeprecation] -> TcRn m Deprecations +rnSrcDeprecDecls [] + = returnM NoDeprecs + +rnSrcDeprecDecls decls + = mappM rn_deprec decls `thenM` \ pairs -> + returnM (DeprecSome (mkNameEnv (catMaybes pairs))) + where + rn_deprec (Deprecation rdr_name txt loc) + = addSrcLoc loc $ + lookupTopSrcBndr rdr_name `thenM` \ name -> + returnM (Just (name, (name,txt))) + +checkModDeprec :: Maybe DeprecTxt -> Deprecations +-- Check for a module deprecation; done once at top level +checkModDeprec Nothing = NoDeprecs +checkModdeprec (Just txt) = DeprecAll txt + +badDeprec d + = sep [ptext SLIT("Illegal deprecation when whole module is deprecated"), + nest 4 (ppr d)] \end{code} %********************************************************* %* * -\subsection{Type declarations} +\subsection{Source code declarations} %* * %********************************************************* -@rnTyDecl@ uses the `global name function' to create a new type -declaration in which local names have been replaced by their original -names, reporting any unknown names. +\begin{code} +rnSrcDecl :: RdrNameHsDecl -> RnM (RenamedHsDecl, FreeVars) -Renaming type variables is a pain. Because they now contain uniques, -it is necessary to pass in an association list which maps a parsed -tyvar to its @Name@ representation. -In some cases (type signatures of values), -it is even necessary to go over the type first -in order to get the set of tyvars used by it, make an assoc list, -and then go over it again to rename the tyvars! -However, we can also do some scoping checks at the same time. +rnSrcDecl (ValD binds) = rnTopBinds binds `thenM` \ (new_binds, fvs) -> + returnM (ValD new_binds, fvs) -\begin{code} -rnDecl (TyClD (TyData new_or_data context tycon tyvars condecls derivings pragmas src_loc)) - = pushSrcLocRn src_loc $ - lookupBndrRn tycon `thenRn` \ tycon' -> - bindTyVarsFVRn data_doc tyvars $ \ tyvars' -> - rnContext data_doc context `thenRn` \ (context', cxt_fvs) -> - checkDupOrQualNames data_doc con_names `thenRn_` - mapFvRn rnConDecl condecls `thenRn` \ (condecls', con_fvs) -> - rnDerivs derivings `thenRn` \ (derivings', deriv_fvs) -> - ASSERT(isNoDataPragmas pragmas) - returnRn (TyClD (TyData new_or_data context' tycon' tyvars' condecls' - derivings' noDataPragmas src_loc), - cxt_fvs `plusFV` con_fvs `plusFV` deriv_fvs) +rnSrcDecl (TyClD tycl_decl) + = rnTyClDecl tycl_decl `thenM` \ new_decl -> + finishSourceTyClDecl tycl_decl new_decl `thenM` \ (new_decl', fvs) -> + returnM (TyClD new_decl', fvs `plusFV` tyClDeclFVs new_decl') + +rnSrcDecl (InstD inst) + = rnInstDecl inst `thenM` \ new_inst -> + finishSourceInstDecl inst new_inst `thenM` \ (new_inst', fvs) -> + returnM (InstD new_inst', fvs `plusFV` instDeclFVs new_inst') + +rnSrcDecl (RuleD rule) + = rnHsRuleDecl rule `thenM` \ (new_rule, fvs) -> + returnM (RuleD new_rule, fvs) + +rnSrcDecl (ForD ford) + = rnHsForeignDecl ford `thenM` \ (new_ford, fvs) -> + returnM (ForD new_ford, fvs) + +rnSrcDecl (DefD (DefaultDecl tys src_loc)) + = addSrcLoc src_loc $ + mapFvRn (rnHsTypeFVs doc_str) tys `thenM` \ (tys', fvs) -> + returnM (DefD (DefaultDecl tys' src_loc), fvs) where - data_doc = text "the data type declaration for" <+> quotes (ppr tycon) - con_names = map conDeclName condecls - -rnDecl (TyClD (TySynonym name tyvars ty src_loc)) - = pushSrcLocRn src_loc $ - lookupBndrRn name `thenRn` \ name' -> - bindTyVarsFVRn syn_doc tyvars $ \ tyvars' -> - rnHsPolyType syn_doc (unquantify ty) `thenRn` \ (ty', ty_fvs) -> - returnRn (TyClD (TySynonym name' tyvars' ty' src_loc), ty_fvs) + doc_str = text "In a `default' declaration" + + +rnSrcDecl (CoreD (CoreDecl name ty rhs loc)) + = addSrcLoc loc $ + lookupTopBndrRn name `thenM` \ name' -> + rnHsTypeFVs doc_str ty `thenM` \ (ty', ty_fvs) -> + rnCoreExpr rhs `thenM` \ rhs' -> + returnM (CoreD (CoreDecl name' ty' rhs' loc), + ty_fvs `plusFV` ufExprFVs rhs') where - syn_doc = text "the declaration for type synonym" <+> quotes (ppr name) - - -- For H98 we do *not* universally quantify on the RHS of a synonym - -- Silently discard context... but the tyvars in the rest won't be in scope - unquantify (HsForAllTy Nothing ctxt ty) | not opt_GlasgowExts = ty - unquantify ty = ty - -rnDecl (TyClD (ClassDecl context cname tyvars fds sigs mbinds pragmas - tname dname dwname snames src_loc)) - = pushSrcLocRn src_loc $ - - lookupBndrRn cname `thenRn` \ cname' -> - - -- Deal with the implicit tycon and datacon name - -- They aren't in scope (because they aren't visible to the user) - -- and what we want to do is simply look them up in the cache; - -- we jolly well ought to get a 'hit' there! - -- So the 'Imported' part of this call is not relevant. - -- Unclean; but since these two are the only place this happens - -- I can't work up the energy to do it more beautifully - mkImportedGlobalFromRdrName tname `thenRn` \ tname' -> - mkImportedGlobalFromRdrName dname `thenRn` \ dname' -> - mkImportedGlobalFromRdrName dwname `thenRn` \ dwname' -> - mapRn mkImportedGlobalFromRdrName snames `thenRn` \ snames' -> - - -- Tyvars scope over bindings and context - bindTyVarsFV2Rn cls_doc tyvars ( \ clas_tyvar_names tyvars' -> + doc_str = text "In the Core declaration for" <+> quotes (ppr name) +\end{code} - -- Check the superclasses - rnContext cls_doc context `thenRn` \ (context', cxt_fvs) -> +%********************************************************* +%* * + Bindings +%* * +%********************************************************* - -- Check the functional dependencies - rnFds cls_doc fds `thenRn` \ (fds', fds_fvs) -> +These chaps are here, rather than in TcBinds, so that there +is just one hi-boot file (for RnSource). rnSrcDecls is part +of the loop too, and it must be defined in this module. - -- Check the signatures - let - -- First process the class op sigs, then the fixity sigs. - (op_sigs, non_op_sigs) = partition isClassOpSig sigs - in - checkDupOrQualNames sig_doc sig_rdr_names_w_locs `thenRn_` - mapFvRn (rn_op cname' clas_tyvar_names fds') op_sigs `thenRn` \ (sigs', sig_fvs) -> - let - binders = mkNameSet [ nm | (ClassOpSig nm _ _ _ _) <- sigs' ] - in - renameSigs (okClsDclSig binders) non_op_sigs `thenRn` \ (non_ops', fix_fvs) -> +\begin{code} +rnTopBinds :: RdrNameHsBinds -> RnM (RenamedHsBinds, FreeVars) +rnTopBinds EmptyBinds = returnM (EmptyBinds, emptyFVs) +rnTopBinds (MonoBind bind sigs _) = rnTopMonoBinds bind sigs + -- The parser doesn't produce other forms + +rnBinds :: RdrNameHsBinds + -> (RenamedHsBinds -> RnM (result, FreeVars)) + -> RnM (result, FreeVars) +rnBinds EmptyBinds thing_inside = thing_inside EmptyBinds +rnBinds (MonoBind bind sigs _) thing_inside = rnMonoBinds bind sigs thing_inside + -- the parser doesn't produce other forms +\end{code} - -- Check the methods - checkDupOrQualNames meth_doc meth_rdr_names_w_locs `thenRn_` - rnMethodBinds mbinds - `thenRn` \ (mbinds', meth_fvs) -> - -- Typechecker is responsible for checking that we only - -- give default-method bindings for things in this class. - -- The renamer *could* check this for class decls, but can't - -- for instance decls. +%********************************************************* +%* * +\subsection{Foreign declarations} +%* * +%********************************************************* - ASSERT(isNoClassPragmas pragmas) - returnRn (TyClD (ClassDecl context' cname' tyvars' fds' (non_ops' ++ sigs') mbinds' - NoClassPragmas tname' dname' dwname' snames' src_loc), - sig_fvs `plusFV` - fix_fvs `plusFV` - cxt_fvs `plusFV` - fds_fvs `plusFV` - meth_fvs - ) - ) +\begin{code} +rnHsForeignDecl (ForeignImport name ty spec isDeprec src_loc) + = addSrcLoc src_loc $ + lookupTopBndrRn name `thenM` \ name' -> + rnHsTypeFVs (fo_decl_msg name) ty `thenM` \ (ty', fvs) -> + returnM (ForeignImport name' ty' spec isDeprec src_loc, + fvs `plusFV` extras spec) where - cls_doc = text "the declaration for class" <+> ppr cname - sig_doc = text "the signatures for class" <+> ppr cname - meth_doc = text "the default-methods for class" <+> ppr cname - - sig_rdr_names_w_locs = [(op,locn) | ClassOpSig op _ _ _ locn <- sigs] - meth_rdr_names_w_locs = bagToList (collectMonoBinders mbinds) - meth_rdr_names = map fst meth_rdr_names_w_locs - - rn_op clas clas_tyvars clas_fds sig@(ClassOpSig op dm_rdr_name explicit_dm ty locn) - = pushSrcLocRn locn $ - lookupBndrRn op `thenRn` \ op_name -> - - -- Check the signature - rnHsSigType (quotes (ppr op)) ty `thenRn` \ (new_ty, op_ty_fvs) -> - let - check_in_op_ty clas_tyvar = - checkRn (clas_tyvar `elemNameSet` oclose clas_fds op_ty_fvs) - (classTyVarNotInOpTyErr clas_tyvar sig) - in - mapRn_ check_in_op_ty clas_tyvars `thenRn_` - - -- Make the default-method name - getModeRn `thenRn` \ mode -> - (case mode of - SourceMode -> -- Source class decl - newImplicitBinder (mkDefaultMethodOcc (rdrNameOcc op)) locn `thenRn` \ dm_name -> - returnRn (dm_name, op `elem` meth_rdr_names, emptyFVs) - - InterfaceMode - -> -- Imported class that has a default method decl - -- See comments with tname, snames, above - lookupImplicitOccRn dm_rdr_name `thenRn` \ dm_name -> - returnRn (dm_name, explicit_dm, if explicit_dm then unitFV dm_name else emptyFVs) - -- An imported class decl for a class decl that had an explicit default - -- method, mentions, rather than defines, - -- the default method, so we must arrange to pull it in - ) `thenRn` \ (dm_name, final_explicit_dm, dm_fvs) -> - - returnRn (ClassOpSig op_name dm_name final_explicit_dm new_ty locn, op_ty_fvs `plusFV` dm_fvs) + extras (CImport _ _ _ _ CWrapper) = mkFVs [newStablePtrName, + bindIOName, returnIOName] + extras _ = emptyFVs + +rnHsForeignDecl (ForeignExport name ty spec isDeprec src_loc) + = addSrcLoc src_loc $ + lookupOccRn name `thenM` \ name' -> + rnHsTypeFVs (fo_decl_msg name) ty `thenM` \ (ty', fvs) -> + returnM (ForeignExport name' ty' spec isDeprec src_loc, + mkFVs [bindIOName, returnIOName] `plusFV` fvs) + +fo_decl_msg name = ptext SLIT("In the foreign declaration for") <+> ppr name \end{code} @@ -278,140 +314,327 @@ rnDecl (TyClD (ClassDecl context cname tyvars fds sigs mbinds pragmas %********************************************************* \begin{code} -rnDecl (InstD (InstDecl inst_ty mbinds uprags dfun_rdr_name src_loc)) - = pushSrcLocRn src_loc $ - rnHsSigType (text "an instance decl") inst_ty `thenRn` \ (inst_ty', inst_fvs) -> +rnInstDecl (InstDecl inst_ty mbinds uprags maybe_dfun_rdr_name src_loc) + -- Used for both source and interface file decls + = addSrcLoc src_loc $ + rnHsSigType (text "an instance decl") inst_ty `thenM` \ inst_ty' -> + + (case maybe_dfun_rdr_name of + Nothing -> returnM Nothing + Just dfun_rdr_name -> lookupGlobalOccRn dfun_rdr_name `thenM` \ dfun_name -> + returnM (Just dfun_name) + ) `thenM` \ maybe_dfun_name -> + + -- The typechecker checks that all the bindings are for the right class. + returnM (InstDecl inst_ty' EmptyMonoBinds [] maybe_dfun_name src_loc) + +-- Compare finishSourceTyClDecl +finishSourceInstDecl (InstDecl _ mbinds uprags _ _ ) + (InstDecl inst_ty _ _ maybe_dfun_name src_loc) + -- Used for both source decls only + = ASSERT( not (maybeToBool maybe_dfun_name) ) -- Source decl! let - inst_tyvars = case inst_ty' of - HsForAllTy (Just inst_tyvars) _ _ -> inst_tyvars - other -> [] + meth_doc = text "In the bindings in an instance declaration" + meth_names = collectLocatedMonoBinders mbinds + (inst_tyvars, _, cls,_) = splitHsInstDeclTy inst_ty -- (Slightly strangely) the forall-d tyvars scope over -- the method bindings too in -- Rename the bindings -- NB meth_names can be qualified! - checkDupNames meth_doc meth_names `thenRn_` - extendTyVarEnvFVRn inst_tyvars ( - rnMethodBinds mbinds - ) `thenRn` \ (mbinds', meth_fvs) -> + checkDupNames meth_doc meth_names `thenM_` + extendTyVarEnvForMethodBinds inst_tyvars ( + rnMethodBinds cls [] mbinds + ) `thenM` \ (mbinds', meth_fvs) -> let - binders = mkNameSet (map fst (bagToList (collectMonoBinders mbinds'))) + binders = collectMonoBinders mbinds' + binder_set = mkNameSet binders in -- Rename the prags and signatures. -- Note that the type variables are not in scope here, -- so that instance Eq a => Eq (T a) where -- {-# SPECIALISE instance Eq a => Eq (T [a]) #-} -- works OK. - renameSigs (okInstDclSig binders) uprags `thenRn` \ (new_uprags, prag_fvs) -> - - getModeRn `thenRn` \ mode -> - (case mode of - InterfaceMode -> lookupImplicitOccRn dfun_rdr_name `thenRn` \ dfun_name -> - returnRn (dfun_name, unitFV dfun_name) - SourceMode -> newDFunName (getDFunKey inst_ty') src_loc - `thenRn` \ dfun_name -> - returnRn (dfun_name, emptyFVs) - ) - `thenRn` \ (dfun_name, dfun_fv) -> - - -- The typechecker checks that all the bindings are for the right class. - returnRn (InstD (InstDecl inst_ty' mbinds' new_uprags dfun_name src_loc), - inst_fvs `plusFV` meth_fvs `plusFV` prag_fvs `plusFV` dfun_fv) - where - meth_doc = text "the bindings in an instance declaration" - meth_names = bagToList (collectMonoBinders mbinds) + -- + -- But the (unqualified) method names are in scope + bindLocalNames binders ( + renameSigsFVs (okInstDclSig binder_set) uprags + ) `thenM` \ (uprags', prag_fvs) -> + + returnM (InstDecl inst_ty mbinds' uprags' maybe_dfun_name src_loc, + meth_fvs `plusFV` prag_fvs) \end{code} %********************************************************* %* * -\subsection{Default declarations} +\subsection{Rules} %* * %********************************************************* \begin{code} -rnDecl (DefD (DefaultDecl tys src_loc)) - = pushSrcLocRn src_loc $ - rnHsTypes doc_str tys `thenRn` \ (tys', fvs) -> - returnRn (DefD (DefaultDecl tys' src_loc), fvs) +rnIfaceRuleDecl (IfaceRule rule_name act vars fn args rhs src_loc) + = addSrcLoc src_loc $ + lookupOccRn fn `thenM` \ fn' -> + rnCoreBndrs vars $ \ vars' -> + mappM rnCoreExpr args `thenM` \ args' -> + rnCoreExpr rhs `thenM` \ rhs' -> + returnM (IfaceRule rule_name act vars' fn' args' rhs' src_loc) + +rnIfaceRuleDecl (IfaceRuleOut fn rule) -- Builtin rules come this way + = lookupOccRn fn `thenM` \ fn' -> + returnM (IfaceRuleOut fn' rule) + +rnHsRuleDecl (HsRule rule_name act vars lhs rhs src_loc) + = addSrcLoc src_loc $ + bindPatSigTyVars (collectRuleBndrSigTys vars) $ + + bindLocalsFVRn doc (map get_var vars) $ \ ids -> + mapFvRn rn_var (vars `zip` ids) `thenM` \ (vars', fv_vars) -> + + rnExpr lhs `thenM` \ (lhs', fv_lhs) -> + rnExpr rhs `thenM` \ (rhs', fv_rhs) -> + let + mb_bad = validRuleLhs ids lhs' + in + checkErr (isNothing mb_bad) + (badRuleLhsErr rule_name lhs' mb_bad) `thenM_` + let + bad_vars = [var | var <- ids, not (var `elemNameSet` fv_lhs)] + in + mappM (addErr . badRuleVar rule_name) bad_vars `thenM_` + returnM (HsRule rule_name act vars' lhs' rhs' src_loc, + fv_vars `plusFV` fv_lhs `plusFV` fv_rhs) where - doc_str = text "a `default' declaration" + doc = text "In the transformation rule" <+> ftext rule_name + + get_var (RuleBndr v) = v + get_var (RuleBndrSig v _) = v + + rn_var (RuleBndr v, id) = returnM (RuleBndr id, emptyFVs) + rn_var (RuleBndrSig v t, id) = rnHsTypeFVs doc t `thenM` \ (t', fvs) -> + returnM (RuleBndrSig id t', fvs) \end{code} -%********************************************************* -%* * -\subsection{Foreign declarations} -%* * -%********************************************************* +Check the shape of a transformation rule LHS. Currently +we only allow LHSs of the form @(f e1 .. en)@, where @f@ is +not one of the @forall@'d variables. We also restrict the form of the LHS so +that it may be plausibly matched. Basically you only get to write ordinary +applications. (E.g. a case expression is not allowed: too elaborate.) + +NB: if you add new cases here, make sure you add new ones to TcRule.ruleLhsTvs \begin{code} -rnDecl (ForD (ForeignDecl name imp_exp ty ext_nm cconv src_loc)) - = pushSrcLocRn src_loc $ - lookupOccRn name `thenRn` \ name' -> - let - ok_ext_nm Dynamic = True - ok_ext_nm (ExtName nm (Just mb)) = isCLabelString nm && isCLabelString mb - ok_ext_nm (ExtName nm Nothing) = isCLabelString nm - - fvs1 = case imp_exp of - FoImport _ | not isDyn -> emptyFVs - FoLabel -> emptyFVs - FoExport | isDyn -> mkNameSet [makeStablePtr_NAME, - deRefStablePtr_NAME, - bindIO_NAME, returnIO_NAME] - | otherwise -> mkNameSet [name'] - _ -> emptyFVs - in - checkRn (ok_ext_nm ext_nm) (badExtName ext_nm) `thenRn_` - rnHsSigType fo_decl_msg ty `thenRn` \ (ty', fvs2) -> - returnRn (ForD (ForeignDecl name' imp_exp ty' ext_nm cconv src_loc), - fvs1 `plusFV` fvs2) - where - fo_decl_msg = ptext SLIT("a foreign declaration") - isDyn = isDynamicExtName ext_nm +validRuleLhs :: [Name] -> RenamedHsExpr -> Maybe RenamedHsExpr +-- Nothing => OK +-- Just e => Not ok, and e is the offending expression +validRuleLhs foralls lhs + = check lhs + where + check (OpApp e1 op _ e2) = check op `seqMaybe` check_e e1 `seqMaybe` check_e e2 + check (HsApp e1 e2) = check e1 `seqMaybe` check_e e2 + check (HsVar v) | v `notElem` foralls = Nothing + check other = Just other -- Failure + + check_e (HsVar v) = Nothing + check_e (HsPar e) = check_e e + check_e (HsLit e) = Nothing + check_e (HsOverLit e) = Nothing + + check_e (OpApp e1 op _ e2) = check_e e1 `seqMaybe` check_e op `seqMaybe` check_e e2 + check_e (HsApp e1 e2) = check_e e1 `seqMaybe` check_e e2 + check_e (NegApp e _) = check_e e + check_e (ExplicitList _ es) = check_es es + check_e (ExplicitTuple es _) = check_es es + check_e other = Just other -- Fails + + check_es es = foldr (seqMaybe . check_e) Nothing es \end{code} + %********************************************************* %* * -\subsection{Rules} +\subsection{Type, class and iface sig declarations} %* * %********************************************************* +@rnTyDecl@ uses the `global name function' to create a new type +declaration in which local names have been replaced by their original +names, reporting any unknown names. + +Renaming type variables is a pain. Because they now contain uniques, +it is necessary to pass in an association list which maps a parsed +tyvar to its @Name@ representation. +In some cases (type signatures of values), +it is even necessary to go over the type first +in order to get the set of tyvars used by it, make an assoc list, +and then go over it again to rename the tyvars! +However, we can also do some scoping checks at the same time. + \begin{code} -rnDecl (RuleD (IfaceRuleDecl var body src_loc)) - = pushSrcLocRn src_loc $ - lookupOccRn var `thenRn` \ var' -> - rnRuleBody body `thenRn` \ (body', fvs) -> - returnRn (RuleD (IfaceRuleDecl var' body' src_loc), fvs `addOneFV` var') +rnTyClDecl (IfaceSig {tcdName = name, tcdType = ty, tcdIdInfo = id_infos, tcdLoc = loc}) + = addSrcLoc loc $ + lookupTopBndrRn name `thenM` \ name' -> + rnHsType doc_str ty `thenM` \ ty' -> + mappM rnIdInfo id_infos `thenM` \ id_infos' -> + returnM (IfaceSig {tcdName = name', tcdType = ty', tcdIdInfo = id_infos', tcdLoc = loc}) + where + doc_str = text "In the interface signature for" <+> quotes (ppr name) + +rnTyClDecl (ForeignType {tcdName = name, tcdFoType = fo_type, tcdExtName = ext_name, tcdLoc = loc}) + = addSrcLoc loc $ + lookupTopBndrRn name `thenM` \ name' -> + returnM (ForeignType {tcdName = name', tcdFoType = fo_type, tcdExtName = ext_name, tcdLoc = loc}) + +rnTyClDecl (TyData {tcdND = new_or_data, tcdCtxt = context, tcdName = tycon, + tcdTyVars = tyvars, tcdCons = condecls, tcdGeneric = want_generic, + tcdDerivs = derivs, tcdLoc = src_loc}) + = addSrcLoc src_loc $ + lookupTopBndrRn tycon `thenM` \ tycon' -> + bindTyVarsRn data_doc tyvars $ \ tyvars' -> + rnContext data_doc context `thenM` \ context' -> + rn_derivs derivs `thenM` \ derivs' -> + checkDupOrQualNames data_doc con_names `thenM_` + + rnConDecls tycon' condecls `thenM` \ condecls' -> + returnM (TyData {tcdND = new_or_data, tcdCtxt = context', tcdName = tycon', + tcdTyVars = tyvars', tcdCons = condecls', tcdGeneric = want_generic, + tcdDerivs = derivs', tcdLoc = src_loc}) + where + data_doc = text "In the data type declaration for" <+> quotes (ppr tycon) + con_names = map conDeclName (visibleDataCons condecls) -rnDecl (RuleD (RuleDecl rule_name tvs vars lhs rhs src_loc)) - = ASSERT( null tvs ) - pushSrcLocRn src_loc $ + rn_derivs Nothing = returnM Nothing + rn_derivs (Just ds) = rnContext data_doc ds `thenM` \ ds' -> returnM (Just ds') + +rnTyClDecl (TySynonym {tcdName = name, tcdTyVars = tyvars, tcdSynRhs = ty, tcdLoc = src_loc}) + = addSrcLoc src_loc $ + lookupTopBndrRn name `thenM` \ name' -> + bindTyVarsRn syn_doc tyvars $ \ tyvars' -> + rnHsType syn_doc ty `thenM` \ ty' -> + returnM (TySynonym {tcdName = name', tcdTyVars = tyvars', tcdSynRhs = ty', tcdLoc = src_loc}) + where + syn_doc = text "In the declaration for type synonym" <+> quotes (ppr name) - bindTyVarsFV2Rn doc (map UserTyVar sig_tvs) $ \ sig_tvs' _ -> - bindLocalsFVRn doc (map get_var vars) $ \ ids -> - mapFvRn rn_var (vars `zip` ids) `thenRn` \ (vars', fv_vars) -> +rnTyClDecl (ClassDecl {tcdCtxt = context, tcdName = cname, + tcdTyVars = tyvars, tcdFDs = fds, tcdSigs = sigs, + tcdLoc = src_loc}) + -- Used for both source and interface file decls + = addSrcLoc src_loc $ + + lookupTopBndrRn cname `thenM` \ cname' -> + + -- Tyvars scope over superclass context and method signatures + bindTyVarsRn cls_doc tyvars $ \ tyvars' -> + + -- Check the superclasses + rnContext cls_doc context `thenM` \ context' -> + + -- Check the functional dependencies + rnFds cls_doc fds `thenM` \ fds' -> - rnExpr lhs `thenRn` \ (lhs', fv_lhs) -> - rnExpr rhs `thenRn` \ (rhs', fv_rhs) -> - checkRn (validRuleLhs ids lhs') - (badRuleLhsErr rule_name lhs') `thenRn_` + -- Check the signatures + -- First process the class op sigs (op_sigs), then the fixity sigs (non_op_sigs). let - bad_vars = [var | var <- ids, not (var `elemNameSet` fv_lhs)] + (op_sigs, non_op_sigs) = partition isClassOpSig sigs + sig_rdr_names_w_locs = [(op,locn) | ClassOpSig op _ _ locn <- sigs] in - mapRn (addErrRn . badRuleVar rule_name) bad_vars `thenRn_` - returnRn (RuleD (RuleDecl rule_name sig_tvs' vars' lhs' rhs' src_loc), - fv_vars `plusFV` fv_lhs `plusFV` fv_rhs) + checkDupOrQualNames sig_doc sig_rdr_names_w_locs `thenM_` + mappM (rnClassOp cname' fds') op_sigs `thenM` \ sigs' -> + let + binders = mkNameSet [ nm | (ClassOpSig nm _ _ _) <- sigs' ] + in + renameSigs (okClsDclSig binders) non_op_sigs `thenM` \ non_ops' -> + + -- Typechecker is responsible for checking that we only + -- give default-method bindings for things in this class. + -- The renamer *could* check this for class decls, but can't + -- for instance decls. + + returnM (ClassDecl { tcdCtxt = context', tcdName = cname', tcdTyVars = tyvars', + tcdFDs = fds', tcdSigs = non_ops' ++ sigs', tcdMeths = Nothing, + tcdLoc = src_loc}) where - doc = text "the transformation rule" <+> ptext rule_name - sig_tvs = extractRuleBndrsTyVars vars - - get_var (RuleBndr v) = v - get_var (RuleBndrSig v _) = v + cls_doc = text "In the declaration for class" <+> ppr cname + sig_doc = text "In the signatures for class" <+> ppr cname + +rnClassOp clas clas_fds sig@(ClassOpSig op dm_stuff ty locn) + = addSrcLoc locn $ + lookupTopBndrRn op `thenM` \ op_name -> + + -- Check the signature + rnHsSigType (quotes (ppr op)) ty `thenM` \ new_ty -> + + -- Make the default-method name + (case dm_stuff of + DefMeth dm_rdr_name + -> -- Imported class that has a default method decl + lookupSysBndr dm_rdr_name `thenM` \ dm_name -> + returnM (DefMeth dm_name) + -- An imported class decl for a class decl that had an explicit default + -- method, mentions, rather than defines, + -- the default method, so we must arrange to pull it in + + GenDefMeth -> returnM GenDefMeth + NoDefMeth -> returnM NoDefMeth + ) `thenM` \ dm_stuff' -> + + returnM (ClassOpSig op_name dm_stuff' new_ty locn) + +finishSourceTyClDecl :: RdrNameTyClDecl -> RenamedTyClDecl -> RnM (RenamedTyClDecl, FreeVars) + -- Used for source file decls only + -- Renames the default-bindings of a class decl +finishSourceTyClDecl (ClassDecl {tcdMeths = Just mbinds, tcdLoc = src_loc}) -- Get mbinds from here + rn_cls_decl@(ClassDecl {tcdName = cls, tcdTyVars = tyvars}) -- Everything else is here + -- There are some default-method bindings (abeit possibly empty) so + -- this is a source-code class declaration + = -- The newLocals call is tiresome: given a generic class decl + -- class C a where + -- op :: a -> a + -- op {| x+y |} (Inl a) = ... + -- op {| x+y |} (Inr b) = ... + -- op {| a*b |} (a*b) = ... + -- we want to name both "x" tyvars with the same unique, so that they are + -- easy to group together in the typechecker. + -- Hence the + addSrcLoc src_loc $ + extendTyVarEnvForMethodBinds tyvars $ + getLocalRdrEnv `thenM` \ name_env -> + let + meth_rdr_names_w_locs = collectLocatedMonoBinders mbinds + gen_rdr_tyvars_w_locs = [(tv,src_loc) | tv <- extractGenericPatTyVars mbinds, + not (tv `elemRdrEnv` name_env)] + in + checkDupOrQualNames meth_doc meth_rdr_names_w_locs `thenM_` + newLocalsRn gen_rdr_tyvars_w_locs `thenM` \ gen_tyvars -> + rnMethodBinds cls gen_tyvars mbinds `thenM` \ (mbinds', meth_fvs) -> + returnM (rn_cls_decl {tcdMeths = Just mbinds'}, meth_fvs) + where + meth_doc = text "In the default-methods for class" <+> ppr (tcdName rn_cls_decl) + +finishSourceTyClDecl _ tycl_decl@(TyData {tcdDerivs = derivings}) + -- Derivings are returned here so that they don't form part of the tyClDeclFVs. + -- This is important, because tyClDeclFVs should contain only the + -- FVs that are `needed' by the interface file declaration, and + -- derivings do not appear in this. It also means that the tcGroups + -- are smaller, which turned out to be important for the usage inference. KSW 2002-02. + = returnM (tycl_decl, + maybe emptyFVs extractHsCtxtTyNames derivings) + +finishSourceTyClDecl _ tycl_decl = returnM (tycl_decl, emptyFVs) + -- Not a class declaration +\end{code} - rn_var (RuleBndr v, id) = returnRn (RuleBndr id, emptyFVs) - rn_var (RuleBndrSig v t, id) = rnHsPolyType doc t `thenRn` \ (t', fvs) -> - returnRn (RuleBndrSig id t', fvs) +For the method bindings in class and instance decls, we extend the +type variable environment iff -fglasgow-exts + +\begin{code} +extendTyVarEnvForMethodBinds tyvars thing_inside + = doptM Opt_GlasgowExts `thenM` \ opt_GlasgowExts -> + if opt_GlasgowExts then + extendTyVarEnvFVRn (map hsTyVarName tyvars) thing_inside + else + thing_inside \end{code} @@ -422,84 +645,63 @@ rnDecl (RuleD (RuleDecl rule_name tvs vars lhs rhs src_loc)) %********************************************************* \begin{code} -rnDerivs :: Maybe [RdrName] -> RnMS (Maybe [Name], FreeVars) - -rnDerivs Nothing -- derivs not specified - = returnRn (Nothing, emptyFVs) - -rnDerivs (Just clss) - = mapRn do_one clss `thenRn` \ clss' -> - returnRn (Just clss', mkNameSet clss') - where - do_one cls = lookupOccRn cls `thenRn` \ clas_name -> - checkRn (getUnique clas_name `elem` derivableClassKeys) - (derivingNonStdClassErr clas_name) `thenRn_` - returnRn clas_name -\end{code} - -\begin{code} conDeclName :: RdrNameConDecl -> (RdrName, SrcLoc) -conDeclName (ConDecl n _ _ _ _ l) = (n,l) - -rnConDecl :: RdrNameConDecl -> RnMS (RenamedConDecl, FreeVars) -rnConDecl (ConDecl name wkr tvs cxt details locn) - = pushSrcLocRn locn $ - checkConName name `thenRn_` - lookupBndrRn name `thenRn` \ new_name -> - - mkImportedGlobalFromRdrName wkr `thenRn` \ new_wkr -> - -- See comments with ClassDecl - - bindTyVarsFVRn doc tvs $ \ new_tyvars -> - rnContext doc cxt `thenRn` \ (new_context, cxt_fvs) -> - rnConDetails doc locn details `thenRn` \ (new_details, det_fvs) -> - returnRn (ConDecl new_name new_wkr new_tyvars new_context new_details locn, - cxt_fvs `plusFV` det_fvs) +conDeclName (ConDecl n _ _ _ l) = (n,l) + +rnConDecls :: Name -> DataConDetails RdrNameConDecl -> RnM (DataConDetails RenamedConDecl) +rnConDecls tycon Unknown = returnM Unknown +rnConDecls tycon (HasCons n) = returnM (HasCons n) +rnConDecls tycon (DataCons condecls) + = -- Check that there's at least one condecl, + -- or else we're reading an interface file, or -fglasgow-exts + (if null condecls then + doptM Opt_GlasgowExts `thenM` \ glaExts -> + getModeRn `thenM` \ mode -> + checkErr (glaExts || isInterfaceMode mode) + (emptyConDeclsErr tycon) + else returnM () + ) `thenM_` + + mappM rnConDecl condecls `thenM` \ condecls' -> + returnM (DataCons condecls') + +rnConDecl :: RdrNameConDecl -> RnM RenamedConDecl +rnConDecl (ConDecl name tvs cxt details locn) + = addSrcLoc locn $ + checkConName name `thenM_` + lookupTopBndrRn name `thenM` \ new_name -> + + bindTyVarsRn doc tvs $ \ new_tyvars -> + rnContext doc cxt `thenM` \ new_context -> + rnConDetails doc locn details `thenM` \ new_details -> + returnM (ConDecl new_name new_tyvars new_context new_details locn) where - doc = text "the definition of data constructor" <+> quotes (ppr name) + doc = text "In the definition of data constructor" <+> quotes (ppr name) -rnConDetails doc locn (VanillaCon tys) - = mapFvRn (rnBangTy doc) tys `thenRn` \ (new_tys, fvs) -> - returnRn (VanillaCon new_tys, fvs) +rnConDetails doc locn (PrefixCon tys) + = mappM (rnBangTy doc) tys `thenM` \ new_tys -> + returnM (PrefixCon new_tys) rnConDetails doc locn (InfixCon ty1 ty2) - = rnBangTy doc ty1 `thenRn` \ (new_ty1, fvs1) -> - rnBangTy doc ty2 `thenRn` \ (new_ty2, fvs2) -> - returnRn (InfixCon new_ty1 new_ty2, fvs1 `plusFV` fvs2) - -rnConDetails doc locn (NewCon ty mb_field) - = rnHsPolyType doc ty `thenRn` \ (new_ty, fvs) -> - rn_field mb_field `thenRn` \ new_mb_field -> - returnRn (NewCon new_ty new_mb_field, fvs) - where - rn_field Nothing = returnRn Nothing - rn_field (Just f) = - lookupBndrRn f `thenRn` \ new_f -> - returnRn (Just new_f) + = rnBangTy doc ty1 `thenM` \ new_ty1 -> + rnBangTy doc ty2 `thenM` \ new_ty2 -> + returnM (InfixCon new_ty1 new_ty2) rnConDetails doc locn (RecCon fields) - = checkDupOrQualNames doc field_names `thenRn_` - mapFvRn (rnField doc) fields `thenRn` \ (new_fields, fvs) -> - returnRn (RecCon new_fields, fvs) + = checkDupOrQualNames doc field_names `thenM_` + mappM (rnField doc) fields `thenM` \ new_fields -> + returnM (RecCon new_fields) where - field_names = [(fld, locn) | (flds, _) <- fields, fld <- flds] - -rnField doc (names, ty) - = mapRn lookupBndrRn names `thenRn` \ new_names -> - rnBangTy doc ty `thenRn` \ (new_ty, fvs) -> - returnRn ((new_names, new_ty), fvs) + field_names = [(fld, locn) | (fld, _) <- fields] -rnBangTy doc (Banged ty) - = rnHsPolyType doc ty `thenRn` \ (new_ty, fvs) -> - returnRn (Banged new_ty, fvs) +rnField doc (name, ty) + = lookupTopBndrRn name `thenM` \ new_name -> + rnBangTy doc ty `thenM` \ new_ty -> + returnM (new_name, new_ty) -rnBangTy doc (Unbanged ty) - = rnHsPolyType doc ty `thenRn` \ (new_ty, fvs) -> - returnRn (Unbanged new_ty, fvs) - -rnBangTy doc (Unpacked ty) - = rnHsPolyType doc ty `thenRn` \ (new_ty, fvs) -> - returnRn (Unpacked new_ty, fvs) +rnBangTy doc (BangType s ty) + = rnHsType doc ty `thenM` \ new_ty -> + returnM (BangType s new_ty) -- This data decl will parse OK -- data T = a Int @@ -512,8 +714,7 @@ rnBangTy doc (Unpacked ty) -- from interface files, which always print in prefix form checkConName name - = checkRn (isRdrDataCon name) - (badDataCon name) + = checkErr (isRdrDataCon name) (badDataCon name) \end{code} @@ -524,197 +725,18 @@ checkConName name %********************************************************* \begin{code} -rnHsSigType :: SDoc -> RdrNameHsType -> RnMS (RenamedHsType, FreeVars) - -- rnHsSigType is used for source-language type signatures, - -- which use *implicit* universal quantification. -rnHsSigType doc_str ty - = rnHsPolyType (text "the type signature for" <+> doc_str) ty - ---------------------------------------- -rnHsPolyType, rnHsType :: SDoc -> RdrNameHsType -> RnMS (RenamedHsType, FreeVars) --- rnHsPolyType is prepared to see a for-all; rnHsType is not --- The former is called for the top level of type sigs and function args. - ---------------------------------------- -rnHsPolyType doc (HsForAllTy Nothing ctxt ty) - -- Implicit quantifiction in source code (no kinds on tyvars) - -- Given the signature C => T we universally quantify - -- over FV(T) \ {in-scope-tyvars} - = getLocalNameEnv `thenRn` \ name_env -> - let - mentioned_in_tau = extractHsTyRdrTyVars ty - forall_tyvars = filter (not . (`elemFM` name_env)) mentioned_in_tau - in - checkConstraints doc forall_tyvars mentioned_in_tau ctxt ty `thenRn` \ ctxt' -> - rnForAll doc (map UserTyVar forall_tyvars) ctxt' ty - -rnHsPolyType doc (HsForAllTy (Just forall_tyvars) ctxt tau) - -- Explicit quantification. - -- Check that the forall'd tyvars are a subset of the - -- free tyvars in the tau-type part - -- That's only a warning... unless the tyvar is constrained by a - -- context in which case it's an error - = let - mentioned_in_tau = extractHsTyRdrTyVars tau - mentioned_in_ctxt = nub [tv | p <- ctxt, - ty <- tys_of_pred p, - tv <- extractHsTyRdrTyVars ty] - tys_of_pred (HsPClass clas tys) = tys - tys_of_pred (HsPIParam n ty) = [ty] - - dubious_guys = filter (`notElem` mentioned_in_tau) forall_tyvar_names - -- dubious = explicitly quantified but not mentioned in tau type - - (bad_guys, warn_guys) = partition (`elem` mentioned_in_ctxt) dubious_guys - -- bad = explicitly quantified and constrained, but not mentioned in tau - -- warn = explicitly quantified but not mentioned in ctxt or tau - - forall_tyvar_names = map getTyVarName forall_tyvars - in - -- mapRn_ (forAllErr doc tau) bad_guys `thenRn_` - mapRn_ (forAllWarn doc tau) warn_guys `thenRn_` - checkConstraints doc forall_tyvar_names mentioned_in_tau ctxt tau `thenRn` \ ctxt' -> - rnForAll doc forall_tyvars ctxt' tau - -rnHsPolyType doc other_ty = rnHsType doc other_ty - - --- Check that each constraint mentions at least one of the forall'd type variables --- Since the forall'd type variables are a subset of the free tyvars --- of the tau-type part, this guarantees that every constraint mentions --- at least one of the free tyvars in ty -checkConstraints doc forall_tyvars tau_vars ctxt ty - = mapRn (checkPred doc forall_tyvars ty) ctxt `thenRn` \ maybe_ctxt' -> - returnRn (catMaybes maybe_ctxt') - -- Remove problem ones, to avoid duplicate error message. - -checkPred doc forall_tyvars ty p@(HsPClass clas tys) - | not_univ = failWithRn Nothing (univErr doc p ty) - | otherwise = returnRn (Just p) - where - ct_vars = extractHsTysRdrTyVars tys - not_univ = -- At least one of the tyvars in each constraint must - -- be universally quantified. This restriction isn't in Hugs - not (any (`elem` forall_tyvars) ct_vars) -checkPred doc forall_tyvars ty p@(HsPIParam _ _) - = returnRn (Just p) - -rnForAll doc forall_tyvars ctxt ty - = bindTyVarsFVRn doc forall_tyvars $ \ new_tyvars -> - rnContext doc ctxt `thenRn` \ (new_ctxt, cxt_fvs) -> - rnHsType doc ty `thenRn` \ (new_ty, ty_fvs) -> - returnRn (mkHsForAllTy (Just new_tyvars) new_ctxt new_ty, - cxt_fvs `plusFV` ty_fvs) - ---------------------------------------- -rnHsType doc ty@(HsForAllTy _ _ inner_ty) - = addWarnRn (unexpectedForAllTy ty) `thenRn_` - rnHsPolyType doc ty - -rnHsType doc (MonoTyVar tyvar) - = lookupOccRn tyvar `thenRn` \ tyvar' -> - returnRn (MonoTyVar tyvar', unitFV tyvar') - -rnHsType doc (MonoFunTy ty1 ty2) - = rnHsPolyType doc ty1 `thenRn` \ (ty1', fvs1) -> - -- Might find a for-all as the arg of a function type - rnHsPolyType doc ty2 `thenRn` \ (ty2', fvs2) -> - -- Or as the result. This happens when reading Prelude.hi - -- when we find return :: forall m. Monad m -> forall a. a -> m a - returnRn (MonoFunTy ty1' ty2', fvs1 `plusFV` fvs2) - -rnHsType doc (MonoListTy ty) - = rnHsType doc ty `thenRn` \ (ty', fvs) -> - returnRn (MonoListTy ty', fvs `addOneFV` listTyCon_name) - --- Unboxed tuples are allowed to have poly-typed arguments. These --- sometimes crop up as a result of CPR worker-wrappering dictionaries. -rnHsType doc (MonoTupleTy tys boxed) - = (if boxed - then mapFvRn (rnHsType doc) tys - else mapFvRn (rnHsPolyType doc) tys) `thenRn` \ (tys', fvs) -> - returnRn (MonoTupleTy tys' boxed, fvs `addOneFV` tup_con_name) - where - tup_con_name = tupleTyCon_name boxed (length tys) - -rnHsType doc (MonoTyApp ty1 ty2) - = rnHsType doc ty1 `thenRn` \ (ty1', fvs1) -> - rnHsType doc ty2 `thenRn` \ (ty2', fvs2) -> - returnRn (MonoTyApp ty1' ty2', fvs1 `plusFV` fvs2) - -rnHsType doc (MonoIParamTy n ty) - = getIPName n `thenRn` \ name -> - rnHsType doc ty `thenRn` \ (ty', fvs) -> - returnRn (MonoIParamTy name ty', fvs) - -rnHsType doc (MonoDictTy clas tys) - = lookupOccRn clas `thenRn` \ clas' -> - rnHsTypes doc tys `thenRn` \ (tys', fvs) -> - returnRn (MonoDictTy clas' tys', fvs `addOneFV` clas') - -rnHsType doc (MonoUsgForAllTy uv_rdr ty) - = bindUVarRn doc uv_rdr $ \ uv_name -> - rnHsType doc ty `thenRn` \ (ty', fvs) -> - returnRn (MonoUsgForAllTy uv_name ty', - fvs ) - -rnHsType doc (MonoUsgTy usg ty) - = newUsg usg `thenRn` \ (usg', usg_fvs) -> - rnHsPolyType doc ty `thenRn` \ (ty', ty_fvs) -> - -- A for-all can occur inside a usage annotation - returnRn (MonoUsgTy usg' ty', - usg_fvs `plusFV` ty_fvs) - where - newUsg usg = case usg of - MonoUsOnce -> returnRn (MonoUsOnce, emptyFVs) - MonoUsMany -> returnRn (MonoUsMany, emptyFVs) - MonoUsVar uv_rdr -> lookupOccRn uv_rdr `thenRn` \ uv_name -> - returnRn (MonoUsVar uv_name, emptyFVs) - -rnHsTypes doc tys = mapFvRn (rnHsType doc) tys -\end{code} - - -\begin{code} -rnContext :: SDoc -> RdrNameContext -> RnMS (RenamedContext, FreeVars) - -rnContext doc ctxt - = mapAndUnzipRn (rnPred doc) ctxt `thenRn` \ (theta, fvs_s) -> - let - (_, dup_asserts) = removeDups (cmpHsPred compare) theta - in - -- Check for duplicate assertions - -- If this isn't an error, then it ought to be: - mapRn_ (addWarnRn . dupClassAssertWarn theta) dup_asserts `thenRn_` - - returnRn (theta, plusFVs fvs_s) - -rnPred doc (HsPClass clas tys) - = lookupOccRn clas `thenRn` \ clas_name -> - rnHsTypes doc tys `thenRn` \ (tys', fvs) -> - returnRn (HsPClass clas_name tys', fvs `addOneFV` clas_name) -rnPred doc (HsPIParam n ty) - = getIPName n `thenRn` \ name -> - rnHsType doc ty `thenRn` \ (ty', fvs) -> - returnRn (HsPIParam name ty', fvs) -\end{code} - -\begin{code} -rnFds :: SDoc -> [([RdrName],[RdrName])] -> RnMS ([([Name],[Name])], FreeVars) +rnFds :: SDoc -> [FunDep RdrName] -> RnM [FunDep Name] rnFds doc fds - = mapAndUnzipRn rn_fds fds `thenRn` \ (theta, fvs_s) -> - returnRn (theta, plusFVs fvs_s) + = mappM rn_fds fds where rn_fds (tys1, tys2) - = rnHsTyVars doc tys1 `thenRn` \ (tys1', fvs1) -> - rnHsTyVars doc tys2 `thenRn` \ (tys2', fvs2) -> - returnRn ((tys1', tys2'), fvs1 `plusFV` fvs2) - -rnHsTyVars doc tvs = mapFvRn (rnHsTyvar doc) tvs -rnHsTyvar doc tyvar - = lookupOccRn tyvar `thenRn` \ tyvar' -> - returnRn (tyvar', unitFV tyvar') + = rnHsTyVars doc tys1 `thenM` \ tys1' -> + rnHsTyVars doc tys2 `thenM` \ tys2' -> + returnM (tys1', tys2') + +rnHsTyVars doc tvs = mappM (rnHsTyvar doc) tvs +rnHsTyvar doc tyvar = lookupOccRn tyvar \end{code} %********************************************************* @@ -724,108 +746,89 @@ rnHsTyvar doc tyvar %********************************************************* \begin{code} -rnIdInfo (HsStrictness str) = returnRn (HsStrictness str, emptyFVs) - -rnIdInfo (HsWorker worker) - = lookupOccRn worker `thenRn` \ worker' -> - returnRn (HsWorker worker', unitFV worker') - -rnIdInfo (HsUnfold inline expr) = rnCoreExpr expr `thenRn` \ (expr', fvs) -> - returnRn (HsUnfold inline expr', fvs) -rnIdInfo (HsArity arity) = returnRn (HsArity arity, emptyFVs) -rnIdInfo (HsUpdate update) = returnRn (HsUpdate update, emptyFVs) -rnIdInfo HsNoCafRefs = returnRn (HsNoCafRefs, emptyFVs) -rnIdInfo HsCprInfo = returnRn (HsCprInfo, emptyFVs) -rnIdInfo (HsSpecialise rule_body) = rnRuleBody rule_body - `thenRn` \ (rule_body', fvs) -> - returnRn (HsSpecialise rule_body', fvs) - -rnRuleBody (UfRuleBody str vars args rhs) - = rnCoreBndrs vars $ \ vars' -> - mapFvRn rnCoreExpr args `thenRn` \ (args', fvs1) -> - rnCoreExpr rhs `thenRn` \ (rhs', fvs2) -> - returnRn (UfRuleBody str vars' args' rhs', fvs1 `plusFV` fvs2) +rnIdInfo (HsWorker worker arity) + = lookupOccRn worker `thenM` \ worker' -> + returnM (HsWorker worker' arity) + +rnIdInfo (HsUnfold inline expr) = rnCoreExpr expr `thenM` \ expr' -> + returnM (HsUnfold inline expr') +rnIdInfo (HsStrictness str) = returnM (HsStrictness str) +rnIdInfo (HsArity arity) = returnM (HsArity arity) +rnIdInfo HsNoCafRefs = returnM HsNoCafRefs \end{code} @UfCore@ expressions. \begin{code} rnCoreExpr (UfType ty) - = rnHsPolyType (text "unfolding type") ty `thenRn` \ (ty', fvs) -> - returnRn (UfType ty', fvs) + = rnHsType (text "unfolding type") ty `thenM` \ ty' -> + returnM (UfType ty') rnCoreExpr (UfVar v) - = lookupOccRn v `thenRn` \ v' -> - returnRn (UfVar v', unitFV v') + = lookupOccRn v `thenM` \ v' -> + returnM (UfVar v') rnCoreExpr (UfLit l) - = returnRn (UfLit l, emptyFVs) + = returnM (UfLit l) rnCoreExpr (UfLitLit l ty) - = rnHsType (text "litlit") ty `thenRn` \ (ty', fvs) -> - returnRn (UfLitLit l ty', fvs) + = rnHsType (text "litlit") ty `thenM` \ ty' -> + returnM (UfLitLit l ty') -rnCoreExpr (UfCCall cc ty) - = rnHsPolyType (text "ccall") ty `thenRn` \ (ty', fvs) -> - returnRn (UfCCall cc ty', fvs) +rnCoreExpr (UfFCall cc ty) + = rnHsType (text "ccall") ty `thenM` \ ty' -> + returnM (UfFCall cc ty') -rnCoreExpr (UfTuple con args) - = lookupOccRn con `thenRn` \ con' -> - mapFvRn rnCoreExpr args `thenRn` \ (args', fvs) -> - returnRn (UfTuple con' args', fvs `addOneFV` con') +rnCoreExpr (UfTuple (HsTupCon boxity arity) args) + = mappM rnCoreExpr args `thenM` \ args' -> + returnM (UfTuple (HsTupCon boxity arity) args') rnCoreExpr (UfApp fun arg) - = rnCoreExpr fun `thenRn` \ (fun', fv1) -> - rnCoreExpr arg `thenRn` \ (arg', fv2) -> - returnRn (UfApp fun' arg', fv1 `plusFV` fv2) + = rnCoreExpr fun `thenM` \ fun' -> + rnCoreExpr arg `thenM` \ arg' -> + returnM (UfApp fun' arg') rnCoreExpr (UfCase scrut bndr alts) - = rnCoreExpr scrut `thenRn` \ (scrut', fvs1) -> - bindCoreLocalFVRn bndr ( \ bndr' -> - mapFvRn rnCoreAlt alts `thenRn` \ (alts', fvs2) -> - returnRn (UfCase scrut' bndr' alts', fvs2) - ) `thenRn` \ (case', fvs3) -> - returnRn (case', fvs1 `plusFV` fvs3) + = rnCoreExpr scrut `thenM` \ scrut' -> + bindCoreLocalRn bndr $ \ bndr' -> + mappM rnCoreAlt alts `thenM` \ alts' -> + returnM (UfCase scrut' bndr' alts') rnCoreExpr (UfNote note expr) - = rnNote note `thenRn` \ (note', fvs1) -> - rnCoreExpr expr `thenRn` \ (expr', fvs2) -> - returnRn (UfNote note' expr', fvs1 `plusFV` fvs2) + = rnNote note `thenM` \ note' -> + rnCoreExpr expr `thenM` \ expr' -> + returnM (UfNote note' expr') rnCoreExpr (UfLam bndr body) = rnCoreBndr bndr $ \ bndr' -> - rnCoreExpr body `thenRn` \ (body', fvs) -> - returnRn (UfLam bndr' body', fvs) + rnCoreExpr body `thenM` \ body' -> + returnM (UfLam bndr' body') rnCoreExpr (UfLet (UfNonRec bndr rhs) body) - = rnCoreExpr rhs `thenRn` \ (rhs', fvs1) -> - rnCoreBndr bndr ( \ bndr' -> - rnCoreExpr body `thenRn` \ (body', fvs2) -> - returnRn (UfLet (UfNonRec bndr' rhs') body', fvs2) - ) `thenRn` \ (result, fvs3) -> - returnRn (result, fvs1 `plusFV` fvs3) + = rnCoreExpr rhs `thenM` \ rhs' -> + rnCoreBndr bndr $ \ bndr' -> + rnCoreExpr body `thenM` \ body' -> + returnM (UfLet (UfNonRec bndr' rhs') body') rnCoreExpr (UfLet (UfRec pairs) body) = rnCoreBndrs bndrs $ \ bndrs' -> - mapFvRn rnCoreExpr rhss `thenRn` \ (rhss', fvs1) -> - rnCoreExpr body `thenRn` \ (body', fvs2) -> - returnRn (UfLet (UfRec (bndrs' `zip` rhss')) body', fvs1 `plusFV` fvs2) + mappM rnCoreExpr rhss `thenM` \ rhss' -> + rnCoreExpr body `thenM` \ body' -> + returnM (UfLet (UfRec (bndrs' `zip` rhss')) body') where (bndrs, rhss) = unzip pairs \end{code} \begin{code} rnCoreBndr (UfValBinder name ty) thing_inside - = rnHsPolyType doc ty `thenRn` \ (ty', fvs1) -> - bindCoreLocalFVRn name ( \ name' -> - thing_inside (UfValBinder name' ty') - ) `thenRn` \ (result, fvs2) -> - returnRn (result, fvs1 `plusFV` fvs2) + = rnHsType doc ty `thenM` \ ty' -> + bindCoreLocalRn name $ \ name' -> + thing_inside (UfValBinder name' ty') where doc = text "unfolding id" rnCoreBndr (UfTyBinder name kind) thing_inside - = bindCoreLocalFVRn name $ \ name' -> + = bindCoreLocalRn name $ \ name' -> thing_inside (UfTyBinder name' kind) rnCoreBndrs [] thing_inside = thing_inside [] @@ -836,56 +839,90 @@ rnCoreBndrs (b:bs) thing_inside = rnCoreBndr b $ \ name' -> \begin{code} rnCoreAlt (con, bndrs, rhs) - = rnUfCon con `thenRn` \ (con', fvs1) -> - bindCoreLocalsFVRn bndrs ( \ bndrs' -> - rnCoreExpr rhs `thenRn` \ (rhs', fvs2) -> - returnRn ((con', bndrs', rhs'), fvs2) - ) `thenRn` \ (result, fvs3) -> - returnRn (result, fvs1 `plusFV` fvs3) + = rnUfCon con `thenM` \ con' -> + bindCoreLocalsRn bndrs $ \ bndrs' -> + rnCoreExpr rhs `thenM` \ rhs' -> + returnM (con', bndrs', rhs') rnNote (UfCoerce ty) - = rnHsPolyType (text "unfolding coerce") ty `thenRn` \ (ty', fvs) -> - returnRn (UfCoerce ty', fvs) + = rnHsType (text "unfolding coerce") ty `thenM` \ ty' -> + returnM (UfCoerce ty') -rnNote (UfSCC cc) = returnRn (UfSCC cc, emptyFVs) -rnNote UfInlineCall = returnRn (UfInlineCall, emptyFVs) -rnNote UfInlineMe = returnRn (UfInlineMe, emptyFVs) +rnNote (UfSCC cc) = returnM (UfSCC cc) +rnNote UfInlineCall = returnM UfInlineCall +rnNote UfInlineMe = returnM UfInlineMe rnUfCon UfDefault - = returnRn (UfDefault, emptyFVs) + = returnM UfDefault + +rnUfCon (UfTupleAlt tup_con) + = returnM (UfTupleAlt tup_con) rnUfCon (UfDataAlt con) - = lookupOccRn con `thenRn` \ con' -> - returnRn (UfDataAlt con', unitFV con') + = lookupOccRn con `thenM` \ con' -> + returnM (UfDataAlt con') rnUfCon (UfLitAlt lit) - = returnRn (UfLitAlt lit, emptyFVs) + = returnM (UfLitAlt lit) rnUfCon (UfLitLitAlt lit ty) - = rnHsPolyType (text "litlit") ty `thenRn` \ (ty', fvs) -> - returnRn (UfLitLitAlt lit ty', fvs) + = rnHsType (text "litlit") ty `thenM` \ ty' -> + returnM (UfLitLitAlt lit ty') \end{code} %********************************************************* -%* * -\subsection{Rule shapes} -%* * +%* * +\subsection{Statistics} +%* * %********************************************************* -Check the shape of a transformation rule LHS. Currently -we only allow LHSs of the form @(f e1 .. en)@, where @f@ is -not one of the @forall@'d variables. - \begin{code} -validRuleLhs foralls lhs - = check lhs +rnStats :: [RenamedHsDecl] -- Imported decls + -> TcRn m () +rnStats imp_decls + = doptM Opt_D_dump_rn_trace `thenM` \ dump_rn_trace -> + doptM Opt_D_dump_rn_stats `thenM` \ dump_rn_stats -> + doptM Opt_D_dump_rn `thenM` \ dump_rn -> + getEps `thenM` \ eps -> + + ioToTcRn (dumpIfSet (dump_rn_trace || dump_rn_stats || dump_rn) + "Renamer statistics" + (getRnStats eps imp_decls)) `thenM_` + returnM () + +getRnStats :: ExternalPackageState -> [RenamedHsDecl] -> SDoc +getRnStats eps imported_decls + = hcat [text "Renamer stats: ", stats] where - check (HsApp e1 e2) = check e1 - check (HsVar v) | v `notElem` foralls = True - check other = False -\end{code} + n_mods = length [() | _ <- moduleEnvElts (eps_PIT eps)] + -- This is really only right for a one-shot compile + (decls_map, n_decls_slurped) = eps_decls eps + + n_decls_left = length [decl | (avail, True, (_,decl)) <- nameEnvElts decls_map + -- Data, newtype, and class decls are in the decls_fm + -- under multiple names; the tycon/class, and each + -- constructor/class op too. + -- The 'True' selects just the 'main' decl + ] + + (insts_left, n_insts_slurped) = eps_insts eps + n_insts_left = length (bagToList insts_left) + + (rules_left, n_rules_slurped) = eps_rules eps + n_rules_left = length (bagToList rules_left) + + stats = vcat + [int n_mods <+> text "interfaces read", + hsep [ int n_decls_slurped, text "type/class/variable imported, out of", + int (n_decls_slurped + n_decls_left), text "read"], + hsep [ int n_insts_slurped, text "instance decls imported, out of", + int (n_insts_slurped + n_insts_left), text "read"], + hsep [ int n_rules_slurped, text "rule decls imported, out of", + int (n_rules_slurped + n_rules_left), text "read"] + ] +\end{code} %********************************************************* %* * @@ -894,81 +931,22 @@ validRuleLhs foralls lhs %********************************************************* \begin{code} -derivingNonStdClassErr clas - = hsep [ptext SLIT("non-standard class"), ppr clas, ptext SLIT("in deriving clause")] - -classTyVarNotInOpTyErr clas_tyvar sig - = hang (hsep [ptext SLIT("Class type variable"), - quotes (ppr clas_tyvar), - ptext SLIT("does not appear in method signature")]) - 4 (ppr sig) - -dupClassAssertWarn ctxt (assertion : dups) - = sep [hsep [ptext SLIT("Duplicate class assertion"), - quotes (pprHsPred assertion), - ptext SLIT("in the context:")], - nest 4 (pprHsContext ctxt <+> ptext SLIT("..."))] - badDataCon name = hsep [ptext SLIT("Illegal data constructor name"), quotes (ppr name)] -forAllWarn doc ty tyvar - | not opt_WarnUnusedMatches = returnRn () - | otherwise - = getModeRn `thenRn` \ mode -> - case mode of { -#ifndef DEBUG - InterfaceMode -> returnRn () ; -- Don't warn of unused tyvars in interface files - -- unless DEBUG is on, in which case it is slightly - -- informative. They can arise from mkRhsTyLam, -#endif -- leading to (say) f :: forall a b. [b] -> [b] - other -> - - addWarnRn ( - sep [ptext SLIT("The universally quantified type variable") <+> quotes (ppr tyvar), - nest 4 (ptext SLIT("does not appear in the type") <+> quotes (ppr ty))] - $$ - (ptext SLIT("In") <+> doc)) - } - -forAllErr doc ty tyvar - = addErrRn ( - sep [ptext SLIT("The constrained type variable") <+> quotes (ppr tyvar), - nest 4 (ptext SLIT("does not appear in the type") <+> quotes (ppr ty))] - $$ - (ptext SLIT("In") <+> doc)) - -univErr doc constraint ty - = sep [ptext SLIT("All of the type variable(s) in the constraint") - <+> quotes (pprHsPred constraint) - <+> ptext SLIT("are already in scope"), - nest 4 (ptext SLIT("At least one must be universally quantified here")) - ] - $$ - (ptext SLIT("In") <+> doc) - -ambigErr doc constraint ty - = sep [ptext SLIT("Ambiguous constraint") <+> quotes (pprHsPred constraint), - nest 4 (ptext SLIT("in the type:") <+> ppr ty), - nest 4 (ptext SLIT("Each forall-d type variable mentioned by the constraint must appear after the =>."))] - $$ - (ptext SLIT("In") <+> doc) - -unexpectedForAllTy ty - = ptext SLIT("Unexpected forall type:") <+> ppr ty - -badRuleLhsErr name lhs - = sep [ptext SLIT("Rule") <+> ptext name <> colon, - nest 4 (ptext SLIT("Illegal left-hand side:") <+> ppr lhs)] +badRuleLhsErr name lhs (Just bad_e) + = sep [ptext SLIT("Rule") <+> ftext name <> colon, + nest 4 (vcat [ptext SLIT("Illegal expression:") <+> ppr bad_e, + ptext SLIT("in left-hand side:") <+> ppr lhs])] $$ ptext SLIT("LHS must be of form (f e1 .. en) where f is not forall'd") badRuleVar name var - = sep [ptext SLIT("Rule") <+> ptext name <> colon, + = sep [ptext SLIT("Rule") <+> doubleQuotes (ftext name) <> colon, ptext SLIT("Forall'd variable") <+> quotes (ppr var) <+> ptext SLIT("does not appear on left hand side")] -badExtName :: ExtName -> Message -badExtName ext_nm - = sep [quotes (ppr ext_nm) <+> ptext SLIT("is not a valid C identifier")] +emptyConDeclsErr tycon + = sep [quotes (ppr tycon) <+> ptext SLIT("has no constructors"), + nest 4 (ptext SLIT("(-fglasgow-exts permits this)"))] \end{code}