Add bang patterns
[ghc-hetmet.git] / ghc / compiler / rename / RnExpr.lhs
index 0d17226..a128c35 100644 (file)
@@ -11,125 +11,46 @@ free variables.
 
 \begin{code}
 module RnExpr (
-       rnMatchGroup, rnMatch, rnGRHSs, rnLExpr, rnExpr, 
-       checkPrecMatch, checkTH
+       rnLExpr, rnExpr, rnStmts
    ) where
 
 #include "HsVersions.h"
 
-import {-# SOURCE #-} RnSource  ( rnSrcDecls, rnBindGroupsAndThen, rnBindGroups, rnSplice ) 
-
---     RnSource imports RnBinds.rnTopMonoBinds, RnExpr.rnExpr
---     RnBinds  imports RnExpr.rnMatch, etc
---     RnExpr   imports [boot] RnSource.rnSrcDecls, RnSource.rnBinds
-
+import RnSource  ( rnSrcDecls, rnSplice, checkTH ) 
+import RnBinds  ( rnLocalBindsAndThen, rnValBinds,
+                  rnMatchGroup, trimWith ) 
 import HsSyn
 import RnHsSyn
 import TcRnMonad
 import RnEnv
 import OccName         ( plusOccEnv )
-import RnNames         ( importsFromLocalDecls )
+import RnNames         ( getLocalDeclBinders, extendRdrEnvRn )
 import RnTypes         ( rnHsTypeFVs, rnLPat, rnOverLit, rnPatsAndThen, rnLit,
-                         dupFieldErr, precParseErr, sectionPrecErr, patSigErr,
-                         checkTupSize )
-import DynFlags        ( DynFlag(..) )
-import BasicTypes      ( Fixity(..), FixityDirection(..), negateFixity, compareFixity )
-import PrelNames       ( hasKey, assertIdKey, assertErrorName,
+                         mkOpFormRn, mkOpAppRn, mkNegAppRn, checkSectionPrec, 
+                         dupFieldErr, checkTupSize )
+import DynFlags                ( DynFlag(..) )
+import BasicTypes      ( FixityDirection(..) )
+import PrelNames       ( thFAKE, hasKey, assertIdKey, assertErrorName,
                          loopAName, choiceAName, appAName, arrAName, composeAName, firstAName,
                          negateName, thenMName, bindMName, failMName )
-import Name            ( Name, nameOccName )
+import Name            ( Name, nameOccName, nameIsLocalOrFrom )
 import NameSet
-import RdrName         ( RdrName )
-import UnicodeUtil     ( stringToUtf8 )
+import RdrName         ( RdrName, emptyGlobalRdrEnv, extendLocalRdrEnv, lookupLocalRdrEnv )
+import LoadIface       ( loadHomeInterface )
 import UniqFM          ( isNullUFM )
 import UniqSet         ( emptyUniqSet )
+import List            ( nub )
 import Util            ( isSingleton )
 import ListSetOps      ( removeDups )
+import Maybes          ( fromJust )
 import Outputable
-import SrcLoc          ( Located(..), unLoc, getLoc, combineLocs, cmpLocated )
+import SrcLoc          ( Located(..), unLoc, getLoc, cmpLocated )
 import FastString
 
 import List            ( unzip4 )
 \end{code}
 
 
-************************************************************************
-*                                                                      *
-\subsection{Match}
-*                                                                      *
-************************************************************************
-
-\begin{code}
-rnMatchGroup :: HsMatchContext Name -> MatchGroup RdrName -> RnM (MatchGroup Name, FreeVars)
-rnMatchGroup ctxt (MatchGroup ms _)
-  = mapFvRn (rnMatch ctxt) ms  `thenM` \ (new_ms, ms_fvs) ->
-    returnM (MatchGroup new_ms placeHolderType, ms_fvs)
-
-rnMatch :: HsMatchContext Name -> LMatch RdrName -> RnM (LMatch Name, FreeVars)
-rnMatch ctxt  = wrapLocFstM (rnMatch' ctxt)
-
-rnMatch' ctxt match@(Match pats maybe_rhs_sig grhss)
-  = 
-       -- Deal with the rhs type signature
-    bindPatSigTyVarsFV rhs_sig_tys     $ 
-    doptM Opt_GlasgowExts              `thenM` \ opt_GlasgowExts ->
-    (case maybe_rhs_sig of
-       Nothing -> returnM (Nothing, emptyFVs)
-       Just ty | opt_GlasgowExts -> rnHsTypeFVs doc_sig ty     `thenM` \ (ty', ty_fvs) ->
-                                    returnM (Just ty', ty_fvs)
-               | otherwise       -> addLocErr ty patSigErr     `thenM_`
-                                    returnM (Nothing, emptyFVs)
-    )                                  `thenM` \ (maybe_rhs_sig', ty_fvs) ->
-
-       -- Now the main event
-    rnPatsAndThen ctxt True pats $ \ pats' ->
-    rnGRHSs ctxt grhss          `thenM` \ (grhss', grhss_fvs) ->
-
-    returnM (Match pats' maybe_rhs_sig' grhss', grhss_fvs `plusFV` ty_fvs)
-       -- The bindPatSigTyVarsFV and rnPatsAndThen will remove the bound FVs
-  where
-     rhs_sig_tys =  case maybe_rhs_sig of
-                       Nothing -> []
-                       Just ty -> [ty]
-     doc_sig = text "In a result type-signature"
-\end{code}
-
-
-%************************************************************************
-%*                                                                     *
-\subsubsection{Guarded right-hand sides (GRHSs)}
-%*                                                                     *
-%************************************************************************
-
-\begin{code}
-rnGRHSs :: HsMatchContext Name -> GRHSs RdrName -> RnM (GRHSs Name, FreeVars)
-
--- gaw 2004
-rnGRHSs ctxt (GRHSs grhss binds)
-  = rnBindGroupsAndThen binds  $ \ binds' ->
-    mapFvRn (rnGRHS ctxt) grhss        `thenM` \ (grhss', fvGRHSs) ->
-    returnM (GRHSs grhss' binds', fvGRHSs)
-
-rnGRHS :: HsMatchContext Name -> LGRHS RdrName -> RnM (LGRHS Name, FreeVars)
-rnGRHS ctxt = wrapLocFstM (rnGRHS' ctxt)
-
-rnGRHS' ctxt (GRHS guards rhs)
-  = do { opt_GlasgowExts <- doptM Opt_GlasgowExts
-       ; checkM (opt_GlasgowExts || is_standard_guard guards)
-                (addWarn (nonStdGuardErr guards))
-
-       ; ((guards', rhs'), fvs) <- rnStmts (PatGuard ctxt) guards $
-                                   rnLExpr rhs
-       ; return (GRHS guards' rhs', fvs) }
-  where
-       -- Standard Haskell 1.4 guards are just a single boolean
-       -- expression, rather than a list of qualifiers as in the
-       -- Glasgow extension
-    is_standard_guard []                     = True
-    is_standard_guard [L _ (ExprStmt _ _ _)] = True
-    is_standard_guard other                 = False
-\end{code}
-
 %************************************************************************
 %*                                                                     *
 \subsubsection{Expressions}
@@ -264,7 +185,7 @@ rnExpr (HsCase expr matches)
     returnM (HsCase new_expr new_matches, e_fvs `plusFV` ms_fvs)
 
 rnExpr (HsLet binds expr)
-  = rnBindGroupsAndThen binds          $ \ binds' ->
+  = rnLocalBindsAndThen binds          $ \ binds' ->
     rnLExpr expr                        `thenM` \ (expr',fvExpr) ->
     returnM (HsLet binds' expr', fvExpr)
 
@@ -334,14 +255,9 @@ Since all the symbols are reservedops we can simply reject them.
 We return a (bogus) EWildPat in each case.
 
 \begin{code}
-rnExpr e@EWildPat = addErr (patSynErr e)       `thenM_`
-                   returnM (EWildPat, emptyFVs)
-
-rnExpr e@(EAsPat _ _) = addErr (patSynErr e)   `thenM_`
-                       returnM (EWildPat, emptyFVs)
-
-rnExpr e@(ELazyPat _) = addErr (patSynErr e)   `thenM_`
-                       returnM (EWildPat, emptyFVs)
+rnExpr e@EWildPat      = patSynErr e
+rnExpr e@(EAsPat {})   = patSynErr e
+rnExpr e@(ELazyPat {}) = patSynErr e
 \end{code}
 
 %************************************************************************
@@ -352,19 +268,25 @@ rnExpr e@(ELazyPat _) = addErr (patSynErr e)      `thenM_`
 
 \begin{code}
 rnExpr (HsProc pat body)
-  = rnPatsAndThen ProcExpr True [pat] $ \ [pat'] ->
-    rnCmdTop body                    `thenM` \ (body',fvBody) ->
+  = newArrowScope $
+    rnPatsAndThen ProcExpr [pat] $ \ [pat'] ->
+    rnCmdTop body               `thenM` \ (body',fvBody) ->
     returnM (HsProc pat' body', fvBody)
 
 rnExpr (HsArrApp arrow arg _ ho rtl)
-  = rnLExpr arrow      `thenM` \ (arrow',fvArrow) ->
-    rnLExpr arg                `thenM` \ (arg',fvArg) ->
+  = select_arrow_scope (rnLExpr arrow) `thenM` \ (arrow',fvArrow) ->
+    rnLExpr arg                                `thenM` \ (arg',fvArg) ->
     returnM (HsArrApp arrow' arg' placeHolderType ho rtl,
             fvArrow `plusFV` fvArg)
+  where
+    select_arrow_scope tc = case ho of
+        HsHigherOrderApp -> tc
+        HsFirstOrderApp  -> escapeArrowScope tc
 
 -- infix form
 rnExpr (HsArrForm op (Just _) [arg1, arg2])
-  = rnLExpr op         `thenM` \ (op'@(L _ (HsVar op_name)),fv_op) ->
+  = escapeArrowScope (rnLExpr op)
+                       `thenM` \ (op'@(L _ (HsVar op_name)),fv_op) ->
     rnCmdTop arg1      `thenM` \ (arg1',fv_arg1) ->
     rnCmdTop arg2      `thenM` \ (arg2',fv_arg2) ->
 
@@ -377,42 +299,12 @@ rnExpr (HsArrForm op (Just _) [arg1, arg2])
              fv_arg1 `plusFV` fv_op `plusFV` fv_arg2)
 
 rnExpr (HsArrForm op fixity cmds)
-  = rnLExpr op         `thenM` \ (op',fvOp) ->
-    rnCmdArgs cmds     `thenM` \ (cmds',fvCmds) ->
+  = escapeArrowScope (rnLExpr op)      `thenM` \ (op',fvOp) ->
+    rnCmdArgs cmds                     `thenM` \ (cmds',fvCmds) ->
     returnM (HsArrForm op' fixity cmds', fvOp `plusFV` fvCmds)
 
 rnExpr other = pprPanic "rnExpr: unexpected expression" (ppr other)
        -- DictApp, DictLam, TyApp, TyLam
-
----------------------------
--- Deal with fixity (cf mkOpAppRn for the method)
-
-mkOpFormRn :: LHsCmdTop Name           -- Left operand; already rearranged
-         -> LHsExpr Name -> Fixity     -- Operator and fixity
-         -> LHsCmdTop Name             -- Right operand (not an infix)
-         -> RnM (HsCmd Name)
-
----------------------------
--- (e11 `op1` e12) `op2` e2
-mkOpFormRn a1@(L loc (HsCmdTop (L _ (HsArrForm op1 (Just fix1) [a11,a12])) _ _ _))
-       op2 fix2 a2
-  | nofix_error
-  = addErr (precParseErr (ppr_op op1,fix1) (ppr_op op2,fix2))  `thenM_`
-    returnM (HsArrForm op2 (Just fix2) [a1, a2])
-
-  | associate_right
-  = mkOpFormRn a12 op2 fix2 a2         `thenM` \ new_c ->
-    returnM (HsArrForm op1 (Just fix1)
-       [a11, L loc (HsCmdTop (L loc new_c) [] placeHolderType [])])
-       -- TODO: locs are wrong
-  where
-    (nofix_error, associate_right) = compareFixity fix1 fix2
-
----------------------------
---     Default case
-mkOpFormRn arg1 op fix arg2                    -- Default case, no rearrangment
-  = returnM (HsArrForm op (Just fix) [arg1, arg2])
-
 \end{code}
 
 
@@ -629,33 +521,56 @@ rnRbinds str rbinds
 %************************************************************************
 
 \begin{code}
-rnBracket (VarBr n) = lookupOccRn n            `thenM` \ name -> 
-                     returnM (VarBr name, unitFV name)
-rnBracket (ExpBr e) = rnLExpr e                `thenM` \ (e', fvs) ->
-                     returnM (ExpBr e', fvs)
-rnBracket (PatBr p) = rnLPat p         `thenM` \ (p', fvs) ->
-                     returnM (PatBr p', fvs)
-rnBracket (TypBr t) = rnHsTypeFVs doc t        `thenM` \ (t', fvs) ->
-                     returnM (TypBr t', fvs)
+rnBracket (VarBr n) = do { name <- lookupOccRn n
+                        ; this_mod <- getModule
+                        ; checkM (nameIsLocalOrFrom this_mod name) $   -- Reason: deprecation checking asumes the
+                          do { loadHomeInterface msg name              -- home interface is loaded, and this is the
+                             ; return () }                             -- only way that is going to happen
+                        ; returnM (VarBr name, unitFV name) }
+                   where
+                     msg = ptext SLIT("Need interface for Template Haskell quoted Name")
+
+rnBracket (ExpBr e) = do { (e', fvs) <- rnLExpr e
+                        ; return (ExpBr e', fvs) }
+rnBracket (PatBr p) = do { (p', fvs) <- rnLPat p
+                        ; return (PatBr p', fvs) }
+rnBracket (TypBr t) = do { (t', fvs) <- rnHsTypeFVs doc t
+                        ; return (TypBr t', fvs) }
                    where
                      doc = ptext SLIT("In a Template-Haskell quoted type")
 rnBracket (DecBr group) 
-  = importsFromLocalDecls group `thenM` \ (rdr_env, avails) ->
-       -- Discard avails (not useful here)
-
-    updGblEnv (\gbl -> gbl { tcg_rdr_env = tcg_rdr_env gbl `plusOccEnv` rdr_env}) $
-       -- Notice plusOccEnv, not plusGlobalRdrEnv.  In this situation we want
-       -- to *shadow* top-level bindings.  E.g.
+  = do         { gbl_env  <- getGblEnv
+
+       ; let gbl_env1 = gbl_env { tcg_mod = thFAKE }
+       -- Note the thFAKE.  The top-level names from the bracketed 
+       -- declarations will go into the name cache, and we don't want them to 
+       -- confuse the Names for the current module.  
+       -- By using a pretend module, thFAKE, we keep them safely out of the way.
+
+       ; names    <- getLocalDeclBinders gbl_env1 group
+       ; rdr_env' <- extendRdrEnvRn emptyGlobalRdrEnv names
+       -- Furthermore, the names in the bracket shouldn't conflict with
+       -- existing top-level names E.g.
        --      foo = 1
        --      bar = [d| foo = 1|]
-       -- So we drop down to plusOccEnv.  (Perhaps there should be a fn in RdrName.)
-
-    rnSrcDecls group   `thenM` \ (tcg_env, group') ->
-       -- Discard the tcg_env; it contains only extra info about fixity
-    let 
-       dus = tcg_dus tcg_env 
-    in
-    returnM (DecBr group', allUses dus)
+       -- But both 'foo's get a LocalDef provenance, so we'd get a complaint unless
+       -- we start with an emptyGlobalRdrEnv
+
+       ; setGblEnv (gbl_env { tcg_rdr_env = tcg_rdr_env gbl_env1 `plusOccEnv` rdr_env',
+                              tcg_dus = emptyDUs }) $ do
+               -- Notice plusOccEnv, not plusGlobalRdrEnv.  In this situation we want
+               -- to *shadow* top-level bindings.  (See the 'foo' example above.)
+               -- If we don't shadow, we'll get an ambiguity complaint when we do 
+               -- a lookupTopBndrRn (which uses lookupGreLocalRn) on the binder of the 'foo'
+               --
+               -- Furthermore, arguably if the splice does define foo, that should hide
+               -- any foo's further out
+               --
+               -- The emptyDUs is so that we just collect uses for this group alone
+
+       { (tcg_env, group') <- rnSrcDecls group
+               -- Discard the tcg_env; it contains only extra info about fixity
+       ; return (DecBr group', allUses (tcg_dus tcg_env)) } }
 \end{code}
 
 %************************************************************************
@@ -704,67 +619,28 @@ rnStmt ctxt (BindStmt pat expr _ _) thing_inside
                -- The binders do not scope over the expression
        ; (bind_op, fvs1) <- lookupSyntaxName bindMName
        ; (fail_op, fvs2) <- lookupSyntaxName failMName
-
-       ; let reportUnused = case ctxt of
-                                ParStmtCtxt{} -> False
-                                _ -> True
-       ; rnPatsAndThen (StmtCtxt ctxt) reportUnused [pat] $ \ [pat'] -> do
+       ; rnPatsAndThen (StmtCtxt ctxt) [pat] $ \ [pat'] -> do
        { (thing, fvs3) <- thing_inside
        ; return ((BindStmt pat' expr' bind_op fail_op, thing),
                  fv_expr `plusFV` fvs1 `plusFV` fvs2 `plusFV` fvs3) }}
-       -- fv_expr shouldn't really be filtered by
-       -- the rnPatsAndThen, but it does not matter
+       -- fv_expr shouldn't really be filtered by the rnPatsAndThen
+       -- but it does not matter because the names are unique
 
 rnStmt ctxt (LetStmt binds) thing_inside
-  = do { checkErr (ok ctxt binds) (badIpBinds binds)
-       ; rnBindGroupsAndThen binds             $ \ binds' -> do
+  = do { checkErr (ok ctxt binds) 
+                  (badIpBinds (ptext SLIT("a parallel list comprehension:")) binds)
+       ; rnLocalBindsAndThen binds             $ \ binds' -> do
        { (thing, fvs) <- thing_inside
        ; return ((LetStmt binds', thing), fvs) }}
   where
        -- We do not allow implicit-parameter bindings in a parallel
        -- list comprehension.  I'm not sure what it might mean.
-    ok (ParStmtCtxt _) binds = not (any is_ip_bind binds)
-    ok _              _     = True
-
-    is_ip_bind (HsIPBinds _) = True
-    is_ip_bind _            = False
-
-rnStmt ctxt (ParStmt stmtss) thing_inside
-  = do { opt_GlasgowExts <- doptM Opt_GlasgowExts
-       ; checkM opt_GlasgowExts parStmtErr
-       ; (stmtss'_w_unit, fv_stmtss) <- mapFvRn rn_branch stmtss
-       ; let
-           bndrss :: [[Name]]  -- NB: Name, not RdrName
-           bndrss        = map (map unLoc . collectLStmtsBinders) stmtss'
-           (bndrs, dups) = removeDups cmpByOcc (concat bndrss)
-           stmtss'       = map fst stmtss'_w_unit
-       ; mappM dupErr dups
-
-       ; bindLocalNamesFV bndrs $ do
-       { (thing, fvs) <- thing_inside
-       -- Note: binders are returned in scope order, so one may
-       --       shadow the next; e.g. x <- xs; x <- ys
-
-       -- Cut down the exported binders to just the ones needed in the body
-       ; let   used_bndrs_s = map (filter (`elemNameSet` fvs)) bndrss
-               unused_bndrs = filter (not . (`elemNameSet` fvs)) bndrs
-
-     -- With processing of the branches and the tail of comprehension done,
-     -- we can finally compute&report any unused ParStmt binders.
-       ; warnUnusedMatches unused_bndrs
-       ; return ((ParStmt (stmtss' `zip` used_bndrs_s), thing),
-                 fv_stmtss `plusFV` fvs) }}
-  where
-    rn_branch (stmts, _) = rnNormalStmts (ParStmtCtxt ctxt) stmts $
-                          return ((), emptyFVs)
-
-    cmpByOcc n1 n2 = nameOccName n1 `compare` nameOccName n2
-    dupErr (v:_) = addErr (ptext SLIT("Duplicate binding in parallel list comprehension for:")
-                           <+> quotes (ppr v))
+    ok (ParStmtCtxt _) (HsIPBinds _) = False
+    ok _              _             = True
 
 rnStmt ctxt (RecStmt rec_stmts _ _ _ _) thing_inside
-  = bindLocatedLocalsRn doc (collectLStmtsBinders rec_stmts)   $ \ _ ->
-    rn_rec_stmts rec_stmts             `thenM` \ segs ->
+  = bindLocatedLocalsRn doc (collectLStmtsBinders rec_stmts)   $ \ bndrs ->
+    rn_rec_stmts bndrs rec_stmts       `thenM` \ segs ->
     thing_inside                       `thenM` \ (thing, fvs) ->
     let
        segs_w_fwd_refs          = addFwdRefs segs
@@ -777,6 +653,59 @@ rnStmt ctxt (RecStmt rec_stmts _ _ _ _) thing_inside
     returnM ((rec_stmt, thing), uses `plusFV` fvs)
   where
     doc = text "In a recursive do statement"
+
+rnStmt ctxt (ParStmt segs) thing_inside
+  = do { opt_GlasgowExts <- doptM Opt_GlasgowExts
+       ; checkM opt_GlasgowExts parStmtErr
+       ; orig_lcl_env <- getLocalRdrEnv
+       ; ((segs',thing), fvs) <- go orig_lcl_env [] segs
+       ; return ((ParStmt segs', thing), fvs) }
+  where
+--  type ParSeg id = [([LStmt id], [id])]
+--  go :: NameSet -> [ParSeg RdrName]
+--       -> RnM (([ParSeg Name], thing), FreeVars)
+
+    go orig_lcl_env bndrs [] 
+       = do { let { (bndrs', dups) = removeDups cmpByOcc bndrs
+                  ; inner_env = extendLocalRdrEnv orig_lcl_env bndrs' }
+            ; mappM dupErr dups
+            ; (thing, fvs) <- setLocalRdrEnv inner_env thing_inside
+            ; return (([], thing), fvs) }
+
+    go orig_lcl_env bndrs_so_far ((stmts, _) : segs)
+       = do { ((stmts', (bndrs, segs', thing)), fvs)
+                 <- rnNormalStmts par_ctxt stmts $ do
+                    {  -- Find the Names that are bound by stmts
+                      lcl_env <- getLocalRdrEnv
+                    ; let { rdr_bndrs = collectLStmtsBinders stmts
+                          ; bndrs = map ( fromJust
+                                        . lookupLocalRdrEnv lcl_env
+                                        . unLoc) rdr_bndrs
+                          ; new_bndrs = nub bndrs ++ bndrs_so_far 
+                               -- The nub is because there might be shadowing
+                               --      x <- e1; x <- e2
+                               -- So we'll look up (Unqual x) twice, getting
+                               -- the second binding both times, which is the
+                       }       -- one we want
+
+                       -- Typecheck the thing inside, passing on all
+                       -- the Names bound, but separately; revert the envt
+                    ; ((segs', thing), fvs) <- setLocalRdrEnv orig_lcl_env $
+                                               go orig_lcl_env new_bndrs segs
+
+                       -- Figure out which of the bound names are used
+                    ; let used_bndrs = filter (`elemNameSet` fvs) bndrs
+                    ; return ((used_bndrs, segs', thing), fvs) }
+
+            ; let seg' = (stmts', bndrs)
+            ; return (((seg':segs'), thing), 
+                      delListFromNameSet fvs bndrs) }
+
+    par_ctxt = ParStmtCtxt ctxt
+
+    cmpByOcc n1 n2 = nameOccName n1 `compare` nameOccName n2
+    dupErr vs = addErr (ptext SLIT("Duplicate binding in parallel list comprehension for:")
+                       <+> quotes (ppr (head vs)))
 \end{code}
 
 
@@ -804,7 +733,7 @@ rnMDoStmts stmts thing_inside
   =    -- Step1: bring all the binders of the mdo into scope
        -- Remember that this also removes the binders from the
        -- finally-returned free-vars
-    bindLocatedLocalsRn doc (collectLStmtsBinders stmts)       $ \ _ ->
+    bindLocatedLocalsRn doc (collectLStmtsBinders stmts)       $ \ bndrs ->
     do { 
        -- Step 2: Rename each individual stmt, making a
        --         singleton segment.  At this stage the FwdRefs field
@@ -812,7 +741,7 @@ rnMDoStmts stmts thing_inside
        --         for which it's the fwd refs within the bind itself
        --         (This set may not be empty, because we're in a recursive 
        --          context.)
-         segs <- rn_rec_stmts stmts
+         segs <- rn_rec_stmts bndrs stmts
 
        ; (thing, fvs_later) <- thing_inside
 
@@ -839,20 +768,24 @@ rnMDoStmts stmts thing_inside
   where
     doc = text "In a recursive mdo-expression"
 
+---------------------------------------------
+rn_rec_stmts :: [Name] -> [LStmt RdrName] -> RnM [Segment (LStmt Name)]
+rn_rec_stmts bndrs stmts = mappM (rn_rec_stmt bndrs) stmts     `thenM` \ segs_s ->
+                          returnM (concat segs_s)
 
 ----------------------------------------------------
-rn_rec_stmt :: LStmt RdrName -> RnM [Segment (LStmt Name)]
+rn_rec_stmt :: [Name] -> LStmt RdrName -> RnM [Segment (LStmt Name)]
        -- Rename a Stmt that is inside a RecStmt (or mdo)
        -- Assumes all binders are already in scope
        -- Turns each stmt into a singleton Stmt
 
-rn_rec_stmt (L loc (ExprStmt expr _ _))
+rn_rec_stmt all_bndrs (L loc (ExprStmt expr _ _))
   = rnLExpr expr               `thenM` \ (expr', fvs) ->
     lookupSyntaxName thenMName `thenM` \ (then_op, fvs1) ->
     returnM [(emptyNameSet, fvs `plusFV` fvs1, emptyNameSet,
              L loc (ExprStmt expr' then_op placeHolderType))]
 
-rn_rec_stmt (L loc (BindStmt pat expr _ _))
+rn_rec_stmt all_bndrs (L loc (BindStmt pat expr _ _))
   = rnLExpr expr               `thenM` \ (expr', fv_expr) ->
     rnLPat pat                 `thenM` \ (pat', fv_pat) ->
     lookupSyntaxName bindMName `thenM` \ (bind_op, fvs1) ->
@@ -864,24 +797,22 @@ rn_rec_stmt (L loc (BindStmt pat expr _ _))
     returnM [(bndrs, fvs, bndrs `intersectNameSet` fvs,
              L loc (BindStmt pat' expr' bind_op fail_op))]
 
-rn_rec_stmt (L loc (LetStmt binds))
-  = rnBindGroups binds         `thenM` \ (binds', du_binds) ->
+rn_rec_stmt all_bndrs (L loc (LetStmt binds@(HsIPBinds _)))
+  = do { addErr (badIpBinds (ptext SLIT("an mdo expression")) binds)
+       ; failM }
+
+rn_rec_stmt all_bndrs (L loc (LetStmt (HsValBinds binds)))
+  = rnValBinds (trimWith all_bndrs) binds      `thenM` \ (binds', du_binds) ->
     returnM [(duDefs du_binds, duUses du_binds, 
-             emptyNameSet, L loc (LetStmt binds'))]
+             emptyNameSet, L loc (LetStmt (HsValBinds binds')))]
 
-rn_rec_stmt (L loc (RecStmt stmts _ _ _ _))    -- Flatten Rec inside Rec
-  = rn_rec_stmts stmts
+rn_rec_stmt all_bndrs (L loc (RecStmt stmts _ _ _ _))  -- Flatten Rec inside Rec
+  = rn_rec_stmts all_bndrs stmts
 
-rn_rec_stmt stmt@(L _ (ParStmt _))     -- Syntactically illegal in mdo
+rn_rec_stmt all_bndrs stmt@(L _ (ParStmt _))   -- Syntactically illegal in mdo
   = pprPanic "rn_rec_stmt" (ppr stmt)
 
 ---------------------------------------------
-rn_rec_stmts :: [LStmt RdrName] -> RnM [Segment (LStmt Name)]
-rn_rec_stmts stmts = mappM rn_rec_stmt stmts   `thenM` \ segs_s ->
-                    returnM (concat segs_s)
-
-
----------------------------------------------
 addFwdRefs :: [Segment a] -> [Segment a]
 -- So far the segments only have forward refs *within* the Stmt
 --     (which happens for bind:  x <- ...x...)
@@ -984,151 +915,6 @@ segsToStmts ((defs, uses, fwds, ss) : segs) fvs_later
 
 %************************************************************************
 %*                                                                     *
-\subsubsection{Precedence Parsing}
-%*                                                                     *
-%************************************************************************
-
-@mkOpAppRn@ deals with operator fixities.  The argument expressions
-are assumed to be already correctly arranged.  It needs the fixities
-recorded in the OpApp nodes, because fixity info applies to the things
-the programmer actually wrote, so you can't find it out from the Name.
-
-Furthermore, the second argument is guaranteed not to be another
-operator application.  Why? Because the parser parses all
-operator appications left-associatively, EXCEPT negation, which
-we need to handle specially.
-
-\begin{code}
-mkOpAppRn :: LHsExpr Name                      -- Left operand; already rearranged
-         -> LHsExpr Name -> Fixity             -- Operator and fixity
-         -> LHsExpr Name                       -- Right operand (not an OpApp, but might
-                                               -- be a NegApp)
-         -> RnM (HsExpr Name)
-
----------------------------
--- (e11 `op1` e12) `op2` e2
-mkOpAppRn e1@(L _ (OpApp e11 op1 fix1 e12)) op2 fix2 e2
-  | nofix_error
-  = addErr (precParseErr (ppr_op op1,fix1) (ppr_op op2,fix2))  `thenM_`
-    returnM (OpApp e1 op2 fix2 e2)
-
-  | associate_right
-  = mkOpAppRn e12 op2 fix2 e2          `thenM` \ new_e ->
-    returnM (OpApp e11 op1 fix1 (L loc' new_e))
-  where
-    loc'= combineLocs e12 e2
-    (nofix_error, associate_right) = compareFixity fix1 fix2
-
----------------------------
---     (- neg_arg) `op` e2
-mkOpAppRn e1@(L _ (NegApp neg_arg neg_name)) op2 fix2 e2
-  | nofix_error
-  = addErr (precParseErr (pp_prefix_minus,negateFixity) (ppr_op op2,fix2))     `thenM_`
-    returnM (OpApp e1 op2 fix2 e2)
-
-  | associate_right
-  = mkOpAppRn neg_arg op2 fix2 e2      `thenM` \ new_e ->
-    returnM (NegApp (L loc' new_e) neg_name)
-  where
-    loc' = combineLocs neg_arg e2
-    (nofix_error, associate_right) = compareFixity negateFixity fix2
-
----------------------------
---     e1 `op` - neg_arg
-mkOpAppRn e1 op1 fix1 e2@(L _ (NegApp neg_arg _))      -- NegApp can occur on the right
-  | not associate_right                                -- We *want* right association
-  = addErr (precParseErr (ppr_op op1, fix1) (pp_prefix_minus, negateFixity))   `thenM_`
-    returnM (OpApp e1 op1 fix1 e2)
-  where
-    (_, associate_right) = compareFixity fix1 negateFixity
-
----------------------------
---     Default case
-mkOpAppRn e1 op fix e2                         -- Default case, no rearrangment
-  = ASSERT2( right_op_ok fix (unLoc e2),
-            ppr e1 $$ text "---" $$ ppr op $$ text "---" $$ ppr fix $$ text "---" $$ ppr e2
-    )
-    returnM (OpApp e1 op fix e2)
-
--- Parser left-associates everything, but 
--- derived instances may have correctly-associated things to
--- in the right operarand.  So we just check that the right operand is OK
-right_op_ok fix1 (OpApp _ _ fix2 _)
-  = not error_please && associate_right
-  where
-    (error_please, associate_right) = compareFixity fix1 fix2
-right_op_ok fix1 other
-  = True
-
--- Parser initially makes negation bind more tightly than any other operator
--- And "deriving" code should respect this (use HsPar if not)
-mkNegAppRn :: LHsExpr id -> SyntaxExpr id -> RnM (HsExpr id)
-mkNegAppRn neg_arg neg_name
-  = ASSERT( not_op_app (unLoc neg_arg) )
-    returnM (NegApp neg_arg neg_name)
-
-not_op_app (OpApp _ _ _ _) = False
-not_op_app other          = True
-\end{code}
-
-\begin{code}
-checkPrecMatch :: Bool -> Name -> MatchGroup Name -> RnM ()
-       -- True indicates an infix lhs
-       -- See comments with rnExpr (OpApp ...) about "deriving"
-
-checkPrecMatch False fn match 
-  = returnM ()
-checkPrecMatch True op (MatchGroup ms _)       
-  = mapM_ check ms                             
-  where
-    check (L _ (Match (p1:p2:_) _ _))
-      = checkPrec op (unLoc p1) False  `thenM_`
-        checkPrec op (unLoc p2) True
-
-    check _ = panic "checkPrecMatch"
-
-checkPrec op (ConPatIn op1 (InfixCon _ _)) right
-  = lookupFixityRn op          `thenM` \  op_fix@(Fixity op_prec  op_dir) ->
-    lookupFixityRn (unLoc op1) `thenM` \ op1_fix@(Fixity op1_prec op1_dir) ->
-    let
-       inf_ok = op1_prec > op_prec || 
-                (op1_prec == op_prec &&
-                 (op1_dir == InfixR && op_dir == InfixR && right ||
-                  op1_dir == InfixL && op_dir == InfixL && not right))
-
-       info  = (ppr_op op,  op_fix)
-       info1 = (ppr_op op1, op1_fix)
-       (infol, infor) = if right then (info, info1) else (info1, info)
-    in
-    checkErr inf_ok (precParseErr infol infor)
-
-checkPrec op pat right
-  = returnM ()
-
--- Check precedence of (arg op) or (op arg) respectively
--- If arg is itself an operator application, then either
---   (a) its precedence must be higher than that of op
---   (b) its precedency & associativity must be the same as that of op
-checkSectionPrec :: FixityDirection -> HsExpr RdrName
-       -> LHsExpr Name -> LHsExpr Name -> RnM ()
-checkSectionPrec direction section op arg
-  = case unLoc arg of
-       OpApp _ op fix _ -> go_for_it (ppr_op op)     fix
-       NegApp _ _       -> go_for_it pp_prefix_minus negateFixity
-       other            -> returnM ()
-  where
-    L _ (HsVar op_name) = op
-    go_for_it pp_arg_op arg_fix@(Fixity arg_prec assoc)
-       = lookupFixityRn op_name        `thenM` \ op_fix@(Fixity op_prec _) ->
-         checkErr (op_prec < arg_prec
-                    || op_prec == arg_prec && direction == assoc)
-                 (sectionPrecErr (ppr_op op_name, op_fix)      
-                 (pp_arg_op, arg_fix) section)
-\end{code}
-
-
-%************************************************************************
-%*                                                                     *
 \subsubsection{Assertion utils}
 %*                                                                     *
 %************************************************************************
@@ -1140,7 +926,7 @@ mkAssertErrorExpr
   = getSrcSpanM                        `thenM` \ sloc ->
     let
        expr = HsApp (L sloc (HsVar assertErrorName)) (L sloc (HsLit msg))
-       msg  = HsStringPrim (mkFastString (stringToUtf8 (showSDoc (ppr sloc))))
+       msg  = HsStringPrim (mkFastString (showSDoc (ppr sloc)))
     in
     returnM (expr, emptyFVs)
 \end{code}
@@ -1152,30 +938,13 @@ mkAssertErrorExpr
 %************************************************************************
 
 \begin{code}
-ppr_op op = quotes (ppr op)    -- Here, op can be a Name or a (Var n), where n is a Name
-pp_prefix_minus = ptext SLIT("prefix `-'")
-
-nonStdGuardErr guard
-  = hang (ptext
-    SLIT("accepting non-standard pattern guards (-fglasgow-exts to suppress this message)")
-    ) 4 (ppr guard)
-
-patSynErr e 
-  = sep [ptext SLIT("Pattern syntax in expression context:"),
-        nest 4 (ppr e)]
-
-#ifdef GHCI 
-checkTH e what = returnM ()    -- OK
-#else
-checkTH e what         -- Raise an error in a stage-1 compiler
-  = addErr (vcat [ptext SLIT("Template Haskell") <+> text what <+>  
-                 ptext SLIT("illegal in a stage-1 compiler"),
-                 nest 2 (ppr e)])
-#endif   
+patSynErr e = do { addErr (sep [ptext SLIT("Pattern syntax in expression context:"),
+                               nest 4 (ppr e)])
+                ; return (EWildPat, emptyFVs) }
 
 parStmtErr = addErr (ptext SLIT("Illegal parallel list comprehension: use -fglasgow-exts"))
 
-badIpBinds binds
-  = hang (ptext SLIT("Implicit-parameter bindings illegal in a parallel list comprehension:")) 4
-        (ppr binds)
+badIpBinds what binds
+  = hang (ptext SLIT("Implicit-parameter bindings illegal in") <+> what)
+        2 (ppr binds)
 \end{code}