fix haddock submodule pointer

[ghc-hetmet.git] / compiler / ghci / ByteCodeGen.lhs
diff --git a/compiler/ghci/ByteCodeGen.lhs b/compiler/ghci/ByteCodeGen.lhs

index 007f3eb..426f4f2 100644 (file)
--- a/compiler/ghci/ByteCodeGen.lhs
+++ b/compiler/ghci/ByteCodeGen.lhs
@@ -5,13 +5,6 @@
  ByteCodeGen: Generate bytecode from Core
  
  \begin{code}
  ByteCodeGen: Generate bytecode from Core
  
  \begin{code}
-{-# OPTIONS -w #-}
--- The above warning supression flag is a temporary kludge.
--- While working on this module you are encouraged to remove it and fix
--- any warnings in the module. See
---     http://hackage.haskell.org/trac/ghc/wiki/Commentary/CodingStyle#Warnings
--- for details
-
  module ByteCodeGen ( UnlinkedBCO, byteCodeGen, coreExprToBCOs ) where
  
  #include "HsVersions.h"
  module ByteCodeGen ( UnlinkedBCO, byteCodeGen, coreExprToBCOs ) where
  
  #include "HsVersions.h"
@@ -20,13 +13,12 @@ import ByteCodeInstr
  import ByteCodeItbls
  import ByteCodeAsm
  import ByteCodeLink
  import ByteCodeItbls
  import ByteCodeAsm
  import ByteCodeLink
-import ByteCodeFFI
  import LibFFI
  
  import Outputable
  import Name
  import LibFFI
  
  import Outputable
  import Name
+import MkId
  import Id
  import Id
-import FiniteMap
  import ForeignCall
  import HscTypes
  import CoreUtils
  import ForeignCall
  import HscTypes
  import CoreUtils
@@ -38,11 +30,7 @@ import CoreFVs
  import Type
  import DataCon
  import TyCon
  import Type
  import DataCon
  import TyCon
-import Class
-import Type
  import Util
  import Util
-import DataCon
-import Var
  import VarSet
  import TysPrim
  import DynFlags
  import VarSet
  import TysPrim
  import DynFlags
@@ -55,39 +43,40 @@ import Bitmap
  import OrdList
  import Constants
  
  import OrdList
  import Constants
  
-import Data.List       ( intersperse, sortBy, zip4, zip6, partition )
+import Data.List
  import Foreign
  import Foreign.C
  import Foreign
  import Foreign.C
-import Control.Exception       ( throwDyn )
  
  
-import GHC.Exts                ( Int(..), ByteArray# )
-
-import Control.Monad   ( when )
-import Data.Char       ( ord, chr )
+import Control.Monad
+import Data.Char
  
  import UniqSupply
  import BreakArray
  import Data.Maybe
  
  import UniqSupply
  import BreakArray
  import Data.Maybe
-import Module 
-import IdInfo 
+import Module
+import IdInfo
+
+import Data.Map (Map)
+import qualified Data.Map as Map
+import qualified FiniteMap as Map
  
  -- -----------------------------------------------------------------------------
  
  -- -----------------------------------------------------------------------------
--- Generating byte code for a complete module 
+-- Generating byte code for a complete module
  
  byteCodeGen :: DynFlags
              -> [CoreBind]
  
  byteCodeGen :: DynFlags
              -> [CoreBind]
-           -> [TyCon]
-            -> ModBreaks 
+            -> [TyCon]
+            -> ModBreaks
              -> IO CompiledByteCode
              -> IO CompiledByteCode
-byteCodeGen dflags binds tycs modBreaks 
+byteCodeGen dflags binds tycs modBreaks
     = do showPass dflags "ByteCodeGen"
  
     = do showPass dflags "ByteCodeGen"
  
-        let flatBinds = [ (bndr, freeVars rhs) 
-                       | (bndr, rhs) <- flattenBinds binds]
+        let flatBinds = [ (bndr, freeVars rhs)
+                        | (bndr, rhs) <- flattenBinds binds]
  
  
-        us <- mkSplitUniqSupply 'y'  
-        (BcM_State _us final_ctr mallocd _, proto_bcos) 
-           <- runBc us modBreaks (mapM schemeTopBind flatBinds)  
+        us <- mkSplitUniqSupply 'y'
+        (BcM_State _us _final_ctr mallocd _, proto_bcos)
+           <- runBc us modBreaks (mapM schemeTopBind flatBinds)
  
          when (notNull mallocd)
               (panic "ByteCodeGen.byteCodeGen: missing final emitBc?")
  
          when (notNull mallocd)
               (panic "ByteCodeGen.byteCodeGen: missing final emitBc?")
@@ -95,28 +84,28 @@ byteCodeGen dflags binds tycs modBreaks
          dumpIfSet_dyn dflags Opt_D_dump_BCOs
             "Proto-BCOs" (vcat (intersperse (char ' ') (map ppr proto_bcos)))
  
          dumpIfSet_dyn dflags Opt_D_dump_BCOs
             "Proto-BCOs" (vcat (intersperse (char ' ') (map ppr proto_bcos)))
  
-        assembleBCOs proto_bcos tycs
-        
+        assembleBCOs dflags proto_bcos tycs
+
  -- -----------------------------------------------------------------------------
  -- Generating byte code for an expression
  
  -- -----------------------------------------------------------------------------
  -- Generating byte code for an expression
  
--- Returns: (the root BCO for this expression, 
+-- Returns: (the root BCO for this expression,
  --           a list of auxilary BCOs resulting from compiling closures)
  coreExprToBCOs :: DynFlags
  --           a list of auxilary BCOs resulting from compiling closures)
  coreExprToBCOs :: DynFlags
-              -> CoreExpr
+               -> CoreExpr
                 -> IO UnlinkedBCO
  coreExprToBCOs dflags expr
   = do showPass dflags "ByteCodeGen"
  
        -- create a totally bogus name for the top-level BCO; this
        -- should be harmless, since it's never used for anything
                 -> IO UnlinkedBCO
  coreExprToBCOs dflags expr
   = do showPass dflags "ByteCodeGen"
  
        -- create a totally bogus name for the top-level BCO; this
        -- should be harmless, since it's never used for anything
-      let invented_name  = mkSystemVarName (mkPseudoUniqueE 0) FSLIT("ExprTopLevel")
+      let invented_name  = mkSystemVarName (mkPseudoUniqueE 0) (fsLit "ExprTopLevel")
            invented_id    = Id.mkLocalId invented_name (panic "invented_id's type")
            invented_id    = Id.mkLocalId invented_name (panic "invented_id's type")
-         
+
        -- the uniques are needed to generate fresh variables when we introduce new
        -- let bindings for ticked expressions
        us <- mkSplitUniqSupply 'y'
        -- the uniques are needed to generate fresh variables when we introduce new
        -- let bindings for ticked expressions
        us <- mkSplitUniqSupply 'y'
-      (BcM_State _us final_ctr mallocd _ , proto_bco)  
+      (BcM_State _us _final_ctr mallocd _ , proto_bco)
           <- runBc us emptyModBreaks (schemeTopBind (invented_id, freeVars expr))
  
        when (notNull mallocd)
           <- runBc us emptyModBreaks (schemeTopBind (invented_id, freeVars expr))
  
        when (notNull mallocd)
@@ -124,7 +113,7 @@ coreExprToBCOs dflags expr
  
        dumpIfSet_dyn dflags Opt_D_dump_BCOs "Proto-BCOs" (ppr proto_bco)
  
  
        dumpIfSet_dyn dflags Opt_D_dump_BCOs "Proto-BCOs" (ppr proto_bco)
  
-      assembleBCO proto_bco
+      assembleBCO dflags proto_bco
  
  
  -- -----------------------------------------------------------------------------
  
  
  -- -----------------------------------------------------------------------------
@@ -132,20 +121,22 @@ coreExprToBCOs dflags expr
  
  type BCInstrList = OrdList BCInstr
  
  
  type BCInstrList = OrdList BCInstr
  
-type Sequel = Int      -- back off to this depth before ENTER
+type Sequel = Word16 -- back off to this depth before ENTER
  
  -- Maps Ids to the offset from the stack _base_ so we don't have
  -- to mess with it after each push/pop.
  
  -- Maps Ids to the offset from the stack _base_ so we don't have
  -- to mess with it after each push/pop.
-type BCEnv = FiniteMap Id Int  -- To find vars on the stack
+type BCEnv = Map Id Word16 -- To find vars on the stack
  
  
+{-
  ppBCEnv :: BCEnv -> SDoc
  ppBCEnv p
     = text "begin-env"
  ppBCEnv :: BCEnv -> SDoc
  ppBCEnv p
     = text "begin-env"
-     $$ nest 4 (vcat (map pp_one (sortBy cmp_snd (fmToList p))))
+     $$ nest 4 (vcat (map pp_one (sortBy cmp_snd (Map.toList p))))
       $$ text "end-env"
       where
          pp_one (var, offset) = int offset <> colon <+> ppr var <+> ppr (idCgRep var)
          cmp_snd x y = compare (snd x) (snd y)
       $$ text "end-env"
       where
          pp_one (var, offset) = int offset <> colon <+> ppr var <+> ppr (idCgRep var)
          cmp_snd x y = compare (snd x) (snd y)
+-}
  
  -- Create a BCO and do a spot of peephole optimisation on the insns
  -- at the same time.
  
  -- Create a BCO and do a spot of peephole optimisation on the insns
  -- at the same time.
@@ -154,41 +145,41 @@ mkProtoBCO
     -> BCInstrList
     -> Either  [AnnAlt Id VarSet] (AnnExpr Id VarSet)
     -> Int
     -> BCInstrList
     -> Either  [AnnAlt Id VarSet] (AnnExpr Id VarSet)
     -> Int
-   -> Int
+   -> Word16
     -> [StgWord]
     -> [StgWord]
-   -> Bool     -- True <=> is a return point, rather than a function
+   -> Bool      -- True <=> is a return point, rather than a function
     -> [BcPtr]
     -> ProtoBCO name
     -> [BcPtr]
     -> ProtoBCO name
-mkProtoBCO nm instrs_ordlist origin arity bitmap_size bitmap is_ret mallocd_blocks 
+mkProtoBCO nm instrs_ordlist origin arity bitmap_size bitmap is_ret mallocd_blocks
     = ProtoBCO {
     = ProtoBCO {
-       protoBCOName = nm,
-       protoBCOInstrs = maybe_with_stack_check,
-       protoBCOBitmap = bitmap,
-       protoBCOBitmapSize = bitmap_size,
-       protoBCOArity = arity,
-       protoBCOExpr = origin,
-       protoBCOPtrs = mallocd_blocks
+        protoBCOName = nm,
+        protoBCOInstrs = maybe_with_stack_check,
+        protoBCOBitmap = bitmap,
+        protoBCOBitmapSize = bitmap_size,
+        protoBCOArity = arity,
+        protoBCOExpr = origin,
+        protoBCOPtrs = mallocd_blocks
        }
       where
          -- Overestimate the stack usage (in words) of this BCO,
          -- and if >= iNTERP_STACK_CHECK_THRESH, add an explicit
          -- stack check.  (The interpreter always does a stack check
          -- for iNTERP_STACK_CHECK_THRESH words at the start of each
        }
       where
          -- Overestimate the stack usage (in words) of this BCO,
          -- and if >= iNTERP_STACK_CHECK_THRESH, add an explicit
          -- stack check.  (The interpreter always does a stack check
          -- for iNTERP_STACK_CHECK_THRESH words at the start of each
-        -- BCO anyway, so we only need to add an explicit on in the
+        -- BCO anyway, so we only need to add an explicit one in the
          -- (hopefully rare) cases when the (overestimated) stack use
          -- exceeds iNTERP_STACK_CHECK_THRESH.
          maybe_with_stack_check
          -- (hopefully rare) cases when the (overestimated) stack use
          -- exceeds iNTERP_STACK_CHECK_THRESH.
          maybe_with_stack_check
-          | is_ret && stack_usage < aP_STACK_SPLIM = peep_d
-               -- don't do stack checks at return points,
-               -- everything is aggregated up to the top BCO
-               -- (which must be a function).
+           | is_ret && stack_usage < fromIntegral aP_STACK_SPLIM = peep_d
+                -- don't do stack checks at return points,
+                -- everything is aggregated up to the top BCO
+                -- (which must be a function).
                  -- That is, unless the stack usage is >= AP_STACK_SPLIM,
                  -- see bug #1466.
                  -- That is, unless the stack usage is >= AP_STACK_SPLIM,
                  -- see bug #1466.
-           | stack_usage >= iNTERP_STACK_CHECK_THRESH
+           | stack_usage >= fromIntegral iNTERP_STACK_CHECK_THRESH
             = STKCHECK stack_usage : peep_d
             | otherwise
             = STKCHECK stack_usage : peep_d
             | otherwise
-           = peep_d    -- the supposedly common case
-             
+           = peep_d     -- the supposedly common case
+
          -- We assume that this sum doesn't wrap
          stack_usage = sum (map bciStackUse peep_d)
  
          -- We assume that this sum doesn't wrap
          stack_usage = sum (map bciStackUse peep_d)
  
@@ -218,19 +209,19 @@ argBits (rep : args)
  schemeTopBind :: (Id, AnnExpr Id VarSet) -> BcM (ProtoBCO Name)
  
  
  schemeTopBind :: (Id, AnnExpr Id VarSet) -> BcM (ProtoBCO Name)
  
  
-schemeTopBind (id, rhs) 
+schemeTopBind (id, rhs)
    | Just data_con <- isDataConWorkId_maybe id,
      isNullaryRepDataCon data_con = do
    | Just data_con <- isDataConWorkId_maybe id,
      isNullaryRepDataCon data_con = do
-       -- Special case for the worker of a nullary data con.
-       -- It'll look like this:        Nil = /\a -> Nil a
-       -- If we feed it into schemeR, we'll get 
-       --      Nil = Nil
-       -- because mkConAppCode treats nullary constructor applications
-       -- by just re-using the single top-level definition.  So
-       -- for the worker itself, we must allocate it directly.
+        -- Special case for the worker of a nullary data con.
+        -- It'll look like this:        Nil = /\a -> Nil a
+        -- If we feed it into schemeR, we'll get
+        --      Nil = Nil
+        -- because mkConAppCode treats nullary constructor applications
+        -- by just re-using the single top-level definition.  So
+        -- for the worker itself, we must allocate it directly.
      -- ioToBc (putStrLn $ "top level BCO")
      emitBc (mkProtoBCO (getName id) (toOL [PACK data_con 0, ENTER])
      -- ioToBc (putStrLn $ "top level BCO")
      emitBc (mkProtoBCO (getName id) (toOL [PACK data_con 0, ENTER])
-                       (Right rhs) 0 0 [{-no bitmap-}] False{-not alts-}) 
+                       (Right rhs) 0 0 [{-no bitmap-}] False{-not alts-})
  
    | otherwise
    = schemeR [{- No free variables -}] (id, rhs)
  
    | otherwise
    = schemeR [{- No free variables -}] (id, rhs)
@@ -246,13 +237,13 @@ schemeTopBind (id, rhs)
  --
  -- Park the resulting BCO in the monad.  Also requires the
  -- variable to which this value was bound, so as to give the
  --
  -- Park the resulting BCO in the monad.  Also requires the
  -- variable to which this value was bound, so as to give the
--- resulting BCO a name. 
+-- resulting BCO a name.
  
  
-schemeR :: [Id]                -- Free vars of the RHS, ordered as they
-                               -- will appear in the thunk.  Empty for
-                               -- top-level things, which have no free vars.
-       -> (Id, AnnExpr Id VarSet)
-       -> BcM (ProtoBCO Name)
+schemeR :: [Id]                 -- Free vars of the RHS, ordered as they
+                                -- will appear in the thunk.  Empty for
+                                -- top-level things, which have no free vars.
+        -> (Id, AnnExpr Id VarSet)
+        -> BcM (ProtoBCO Name)
  schemeR fvs (nm, rhs)
  {-
     | trace (showSDoc (
  schemeR fvs (nm, rhs)
  {-
     | trace (showSDoc (
@@ -264,62 +255,65 @@ schemeR fvs (nm, rhs)
     = undefined
     | otherwise
  -}
     = undefined
     | otherwise
  -}
-   = schemeR_wrk fvs nm rhs (collect [] rhs)
+   = schemeR_wrk fvs nm rhs (collect rhs)
  
  
-collect :: [Var] -> AnnExpr Id VarSet -> ([Var], AnnExpr' Id VarSet)
-collect xs (_, AnnNote note e) = collect xs e
-collect xs (_, AnnCast e _)    = collect xs e
-collect xs (_, AnnLam x e)     = collect (if isTyVar x then xs else (x:xs)) e
-collect xs (_, not_lambda)     = (reverse xs, not_lambda)
+collect :: AnnExpr Id VarSet -> ([Var], AnnExpr' Id VarSet)
+collect (_, e) = go [] e
+  where
+    go xs e | Just e' <- bcView e = go xs e'
+    go xs (AnnLam x (_,e))        = go (x:xs) e
+    go xs not_lambda              = (reverse xs, not_lambda)
  
  
-schemeR_wrk :: [Id] -> Id -> AnnExpr Id VarSet -> ([Var], AnnExpr' Var VarSet) -> BcM (ProtoBCO Name) 
+schemeR_wrk :: [Id] -> Id -> AnnExpr Id VarSet -> ([Var], AnnExpr' Var VarSet) -> BcM (ProtoBCO Name)
  schemeR_wrk fvs nm original_body (args, body)
  schemeR_wrk fvs nm original_body (args, body)
-   = let 
-        all_args  = reverse args ++ fvs
-        arity     = length all_args
-        -- all_args are the args in reverse order.  We're compiling a function
-        -- \fv1..fvn x1..xn -> e 
-        -- i.e. the fvs come first
-
-         szsw_args = map idSizeW all_args
+   = let
+         all_args  = reverse args ++ fvs
+         arity     = length all_args
+         -- all_args are the args in reverse order.  We're compiling a function
+         -- \fv1..fvn x1..xn -> e
+         -- i.e. the fvs come first
+
+         szsw_args = map (fromIntegral . idSizeW) all_args
           szw_args  = sum szsw_args
           szw_args  = sum szsw_args
-         p_init    = listToFM (zip all_args (mkStackOffsets 0 szsw_args))
+         p_init    = Map.fromList (zip all_args (mkStackOffsets 0 szsw_args))
  
  
-        -- make the arg bitmap
-        bits = argBits (reverse (map idCgRep all_args))
-        bitmap_size = length bits
-        bitmap = mkBitmap bits
+         -- make the arg bitmap
+         bits = argBits (reverse (map idCgRep all_args))
+         bitmap_size = genericLength bits
+         bitmap = mkBitmap bits
       in do
       in do
-     body_code <- schemeER_wrk szw_args p_init body   
- 
+     body_code <- schemeER_wrk szw_args p_init body
+
       emitBc (mkProtoBCO (getName nm) body_code (Right original_body)
       emitBc (mkProtoBCO (getName nm) body_code (Right original_body)
-               arity bitmap_size bitmap False{-not alts-})
+                 arity bitmap_size bitmap False{-not alts-})
  
  -- introduce break instructions for ticked expressions
  
  -- introduce break instructions for ticked expressions
-schemeER_wrk :: Int -> BCEnv -> AnnExpr' Id VarSet -> BcM BCInstrList
+schemeER_wrk :: Word16 -> BCEnv -> AnnExpr' Id VarSet -> BcM BCInstrList
  schemeER_wrk d p rhs
  schemeER_wrk d p rhs
-   | Just (tickInfo, (_annot, newRhs)) <- isTickedExp' rhs = do 
-        code <- schemeE d 0 p newRhs 
-        arr <- getBreakArray 
-        let idOffSets = getVarOffSets d p tickInfo 
+   | Just (tickInfo, (_annot, newRhs)) <- isTickedExp' rhs = do
+        code <- schemeE d 0 p newRhs
+        arr <- getBreakArray
+        let idOffSets = getVarOffSets d p tickInfo
          let tickNumber = tickInfo_number tickInfo
          let tickNumber = tickInfo_number tickInfo
-        let breakInfo = BreakInfo 
+        let breakInfo = BreakInfo
                          { breakInfo_module = tickInfo_module tickInfo
                          { breakInfo_module = tickInfo_module tickInfo
-                        , breakInfo_number = tickNumber 
+                        , breakInfo_number = tickNumber
                          , breakInfo_vars = idOffSets
                          , breakInfo_resty = exprType (deAnnotate' newRhs)
                          }
                          , breakInfo_vars = idOffSets
                          , breakInfo_resty = exprType (deAnnotate' newRhs)
                          }
-        let breakInstr = case arr of (BA arr#) -> BRK_FUN arr# tickNumber breakInfo 
+        let breakInstr = case arr of
+                         BA arr# ->
+                             BRK_FUN arr# (fromIntegral tickNumber) breakInfo
          return $ breakInstr `consOL` code
          return $ breakInstr `consOL` code
-   | otherwise = schemeE d 0 p rhs 
+   | otherwise = schemeE d 0 p rhs
  
  
-getVarOffSets :: Int -> BCEnv -> TickInfo -> [(Id, Int)]
-getVarOffSets d p = catMaybes . map (getOffSet d p) . tickInfo_locals 
+getVarOffSets :: Word16 -> BCEnv -> TickInfo -> [(Id, Word16)]
+getVarOffSets d p = catMaybes . map (getOffSet d p) . tickInfo_locals
  
  
-getOffSet :: Int -> BCEnv -> Id -> Maybe (Id, Int)
-getOffSet d env id 
-   = case lookupBCEnv_maybe env id of
-        Nothing     -> Nothing 
+getOffSet :: Word16 -> BCEnv -> Id -> Maybe (Id, Word16)
+getOffSet d env id
+   = case lookupBCEnv_maybe id env of
+        Nothing     -> Nothing
          Just offset -> Just (id, d - offset)
  
  fvsToEnv :: BCEnv -> VarSet -> [Id]
          Just offset -> Just (id, d - offset)
  
  fvsToEnv :: BCEnv -> VarSet -> [Id]
@@ -331,31 +325,35 @@ fvsToEnv :: BCEnv -> VarSet -> [Id]
  --
  -- The code that constructs the thunk, and the code that executes
  -- it, have to agree about this layout
  --
  -- The code that constructs the thunk, and the code that executes
  -- it, have to agree about this layout
-fvsToEnv p fvs = [v | v <- varSetElems fvs, 
-                     isId v,           -- Could be a type variable
-                     v `elemFM` p]
+fvsToEnv p fvs = [v | v <- varSetElems fvs,
+                      isId v,           -- Could be a type variable
+                      v `Map.member` p]
  
  -- -----------------------------------------------------------------------------
  -- schemeE
  
  
  -- -----------------------------------------------------------------------------
  -- schemeE
  
-data TickInfo 
-   = TickInfo   
+data TickInfo
+   = TickInfo
       { tickInfo_number :: Int     -- the (module) unique number of the tick
       { tickInfo_number :: Int     -- the (module) unique number of the tick
-     , tickInfo_module :: Module  -- the origin of the ticked expression 
+     , tickInfo_module :: Module  -- the origin of the ticked expression
       , tickInfo_locals :: [Id]    -- the local vars in scope at the ticked expression
       , tickInfo_locals :: [Id]    -- the local vars in scope at the ticked expression
-     } 
+     }
  
  instance Outputable TickInfo where
  
  instance Outputable TickInfo where
-   ppr info = text "TickInfo" <+> 
+   ppr info = text "TickInfo" <+>
                parens (int (tickInfo_number info) <+> ppr (tickInfo_module info) <+>
                        ppr (tickInfo_locals info))
  
  -- Compile code to apply the given expression to the remaining args
  -- on the stack, returning a HNF.
                parens (int (tickInfo_number info) <+> ppr (tickInfo_module info) <+>
                        ppr (tickInfo_locals info))
  
  -- Compile code to apply the given expression to the remaining args
  -- on the stack, returning a HNF.
-schemeE :: Int -> Sequel -> BCEnv -> AnnExpr' Id VarSet -> BcM BCInstrList
+schemeE :: Word16 -> Sequel -> BCEnv -> AnnExpr' Id VarSet -> BcM BCInstrList
+
+schemeE d s p e
+   | Just e' <- bcView e
+   = schemeE d s p e'
  
  -- Delegate tail-calls to schemeT.
  
  -- Delegate tail-calls to schemeT.
-schemeE d s p e@(AnnApp f a) 
+schemeE d s p e@(AnnApp _ _)
     = schemeT d s p e
  
  schemeE d s p e@(AnnVar v)
     = schemeT d s p e
  
  schemeE d s p e@(AnnVar v)
@@ -364,12 +362,12 @@ schemeE d s p e@(AnnVar v)
       schemeT d s p e
  
     | otherwise
       schemeT d s p e
  
     | otherwise
-   = do -- Returning an unlifted value.  
+   = do -- Returning an unlifted value.
          -- Heave it on the stack, SLIDE, and RETURN.
          (push, szw) <- pushAtom d p (AnnVar v)
          -- Heave it on the stack, SLIDE, and RETURN.
          (push, szw) <- pushAtom d p (AnnVar v)
-        return (push                   -- value onto stack
-                  `appOL`  mkSLIDE szw (d-s)   -- clear to sequel
-                  `snocOL` RETURN_UBX v_rep)   -- go
+        return (push                       -- value onto stack
+                `appOL`  mkSLIDE szw (d-s) -- clear to sequel
+                `snocOL` RETURN_UBX v_rep) -- go
     where
        v_type = idType v
        v_rep = typeCgRep v_type
     where
        v_type = idType v
        v_rep = typeCgRep v_type
@@ -377,19 +375,19 @@ schemeE d s p e@(AnnVar v)
  schemeE d s p (AnnLit literal)
     = do (push, szw) <- pushAtom d p (AnnLit literal)
          let l_rep = typeCgRep (literalType literal)
  schemeE d s p (AnnLit literal)
     = do (push, szw) <- pushAtom d p (AnnLit literal)
          let l_rep = typeCgRep (literalType literal)
-        return (push                   -- value onto stack
-               `appOL`  mkSLIDE szw (d-s)      -- clear to sequel
-               `snocOL` RETURN_UBX l_rep)      -- go
+        return (push                       -- value onto stack
+                `appOL`  mkSLIDE szw (d-s) -- clear to sequel
+                `snocOL` RETURN_UBX l_rep) -- go
  
  schemeE d s p (AnnLet (AnnNonRec x (_,rhs)) (_,body))
     | (AnnVar v, args_r_to_l) <- splitApp rhs,
       Just data_con <- isDataConWorkId_maybe v,
       dataConRepArity data_con == length args_r_to_l
  
  schemeE d s p (AnnLet (AnnNonRec x (_,rhs)) (_,body))
     | (AnnVar v, args_r_to_l) <- splitApp rhs,
       Just data_con <- isDataConWorkId_maybe v,
       dataConRepArity data_con == length args_r_to_l
-   = do        -- Special case for a non-recursive let whose RHS is a 
-       -- saturatred constructor application.
-       -- Just allocate the constructor and carry on
+   = do -- Special case for a non-recursive let whose RHS is a
+        -- saturatred constructor application.
+        -- Just allocate the constructor and carry on
          alloc_code <- mkConAppCode d s p data_con args_r_to_l
          alloc_code <- mkConAppCode d s p data_con args_r_to_l
-        body_code <- schemeE (d+1) s (addToFM p x d) body
+        body_code <- schemeE (d+1) s (Map.insert x d p) body
          return (alloc_code `appOL` body_code)
  
  -- General case for let.  Generates correct, if inefficient, code in
          return (alloc_code `appOL` body_code)
  
  -- General case for let.  Generates correct, if inefficient, code in
@@ -397,54 +395,54 @@ schemeE d s p (AnnLet (AnnNonRec x (_,rhs)) (_,body))
  schemeE d s p (AnnLet binds (_,body))
     = let (xs,rhss) = case binds of AnnNonRec x rhs  -> ([x],[rhs])
                                     AnnRec xs_n_rhss -> unzip xs_n_rhss
  schemeE d s p (AnnLet binds (_,body))
     = let (xs,rhss) = case binds of AnnNonRec x rhs  -> ([x],[rhs])
                                     AnnRec xs_n_rhss -> unzip xs_n_rhss
-         n_binds = length xs
+         n_binds = genericLength xs
  
           fvss  = map (fvsToEnv p' . fst) rhss
  
           -- Sizes of free vars
  
           fvss  = map (fvsToEnv p' . fst) rhss
  
           -- Sizes of free vars
-         sizes = map (\rhs_fvs -> sum (map idSizeW rhs_fvs)) fvss
+         sizes = map (\rhs_fvs -> sum (map (fromIntegral . idSizeW) rhs_fvs)) fvss
  
  
-        -- the arity of each rhs
-        arities = map (length . fst . collect []) rhss
+         -- the arity of each rhs
+         arities = map (genericLength . fst . collect) rhss
  
           -- This p', d' defn is safe because all the items being pushed
           -- are ptrs, so all have size 1.  d' and p' reflect the stack
           -- after the closures have been allocated in the heap (but not
           -- filled in), and pointers to them parked on the stack.
  
           -- This p', d' defn is safe because all the items being pushed
           -- are ptrs, so all have size 1.  d' and p' reflect the stack
           -- after the closures have been allocated in the heap (but not
           -- filled in), and pointers to them parked on the stack.
-         p'    = addListToFM p (zipE xs (mkStackOffsets d (nOfThem n_binds 1)))
+         p'    = Map.insertList (zipE xs (mkStackOffsets d (genericReplicate n_binds 1))) p
           d'    = d + n_binds
           zipE  = zipEqual "schemeE"
  
           -- ToDo: don't build thunks for things with no free variables
           d'    = d + n_binds
           zipE  = zipEqual "schemeE"
  
           -- ToDo: don't build thunks for things with no free variables
-         build_thunk dd [] size bco off arity
+         build_thunk _ [] size bco off arity
              = return (PUSH_BCO bco `consOL` unitOL (mkap (off+size) size))
              = return (PUSH_BCO bco `consOL` unitOL (mkap (off+size) size))
-          where 
-               mkap | arity == 0 = MKAP
-                    | otherwise  = MKPAP
+           where
+                mkap | arity == 0 = MKAP
+                     | otherwise  = MKPAP
           build_thunk dd (fv:fvs) size bco off arity = do
           build_thunk dd (fv:fvs) size bco off arity = do
-              (push_code, pushed_szw) <- pushAtom dd p' (AnnVar fv) 
+              (push_code, pushed_szw) <- pushAtom dd p' (AnnVar fv)
                more_push_code <- build_thunk (dd+pushed_szw) fvs size bco off arity
                return (push_code `appOL` more_push_code)
  
           alloc_code = toOL (zipWith mkAlloc sizes arities)
                more_push_code <- build_thunk (dd+pushed_szw) fvs size bco off arity
                return (push_code `appOL` more_push_code)
  
           alloc_code = toOL (zipWith mkAlloc sizes arities)
-          where mkAlloc sz 0
+           where mkAlloc sz 0
                      | is_tick     = ALLOC_AP_NOUPD sz
                      | otherwise   = ALLOC_AP sz
                      | is_tick     = ALLOC_AP_NOUPD sz
                      | otherwise   = ALLOC_AP sz
-                mkAlloc sz arity = ALLOC_PAP arity sz
+                 mkAlloc sz arity = ALLOC_PAP arity sz
  
  
-         is_tick = case binds of 
+         is_tick = case binds of
                       AnnNonRec id _ -> occNameFS (getOccName id) == tickFS
                       _other -> False
  
                       AnnNonRec id _ -> occNameFS (getOccName id) == tickFS
                       _other -> False
  
-        compile_bind d' fvs x rhs size arity off = do
-               bco <- schemeR fvs (x,rhs)
-               build_thunk d' fvs size bco off arity
+         compile_bind d' fvs x rhs size arity off = do
+                bco <- schemeR fvs (x,rhs)
+                build_thunk d' fvs size bco off arity
  
  
-        compile_binds = 
-           [ compile_bind d' fvs x rhs size arity n
-           | (fvs, x, rhs, size, arity, n) <- 
-               zip6 fvss xs rhss sizes arities [n_binds, n_binds-1 .. 1]
-           ]
+         compile_binds =
+            [ compile_bind d' fvs x rhs size arity n
+            | (fvs, x, rhs, size, arity, n) <-
+                zip6 fvss xs rhss sizes arities [n_binds, n_binds-1 .. 1]
+            ]
       in do
       body_code <- schemeE d' s p' body
       thunk_codes <- sequence compile_binds
       in do
       body_code <- schemeE d' s p' body
       thunk_codes <- sequence compile_binds
@@ -457,9 +455,21 @@ schemeE d s p (AnnLet binds (_,body))
  -- best way to calculate the free vars but it seemed like the least
  -- intrusive thing to do
  schemeE d s p exp@(AnnCase {})
  -- best way to calculate the free vars but it seemed like the least
  -- intrusive thing to do
  schemeE d s p exp@(AnnCase {})
-   | Just (tickInfo,rhs) <- isTickedExp' exp
+   | Just (_tickInfo, _rhs) <- isTickedExp' exp
     = if isUnLiftedType ty
     = if isUnLiftedType ty
-        then schemeE d s p (snd rhs)
+        then do
+          -- If the result type is unlifted, then we must generate
+          --   let f = \s . case tick# of _ -> e
+          --   in  f realWorld#
+          -- When we stop at the breakpoint, _result will have an unlifted
+          -- type and hence won't be bound in the environment, but the
+          -- breakpoint will otherwise work fine.
+          id <- newId (mkFunTy realWorldStatePrimTy ty)
+          st <- newId realWorldStatePrimTy
+          let letExp = AnnLet (AnnNonRec id (fvs, AnnLam st (emptyVarSet, exp)))
+                              (emptyVarSet, (AnnApp (emptyVarSet, AnnVar id)
+                                                    (emptyVarSet, AnnVar realWorldPrimId)))
+          schemeE d s p letExp
          else do
            id <- newId ty
            -- Todo: is emptyVarSet correct on the next line?
          else do
            id <- newId ty
            -- Todo: is emptyVarSet correct on the next line?
@@ -469,50 +479,44 @@ schemeE d s p exp@(AnnCase {})
           fvs  = exprFreeVars exp'
           ty   = exprType exp'
  
           fvs  = exprFreeVars exp'
           ty   = exprType exp'
  
-schemeE d s p (AnnCase scrut bndr _ [(DataAlt dc, [bind1, bind2], rhs)])
+schemeE d s p (AnnCase scrut _ _ [(DataAlt dc, [bind1, bind2], rhs)])
     | isUnboxedTupleCon dc, VoidArg <- typeCgRep (idType bind1)
     | isUnboxedTupleCon dc, VoidArg <- typeCgRep (idType bind1)
-       -- Convert 
-       --      case .... of x { (# VoidArg'd-thing, a #) -> ... }
-       -- to
-       --      case .... of a { DEFAULT -> ... }
-       -- becuse the return convention for both are identical.
-       --
-       -- Note that it does not matter losing the void-rep thing from the
-       -- envt (it won't be bound now) because we never look such things up.
+        -- Convert
+        --      case .... of x { (# VoidArg'd-thing, a #) -> ... }
+        -- to
+        --      case .... of a { DEFAULT -> ... }
+        -- becuse the return convention for both are identical.
+        --
+        -- Note that it does not matter losing the void-rep thing from the
+        -- envt (it won't be bound now) because we never look such things up.
  
     = --trace "automagic mashing of case alts (# VoidArg, a #)" $
  
     = --trace "automagic mashing of case alts (# VoidArg, a #)" $
-     doCase d s p scrut bind2 [(DEFAULT, [], rhs)] True{-unboxed tuple-} 
+     doCase d s p scrut bind2 [(DEFAULT, [], rhs)] True{-unboxed tuple-}
  
     | isUnboxedTupleCon dc, VoidArg <- typeCgRep (idType bind2)
     = --trace "automagic mashing of case alts (# a, VoidArg #)" $
  
     | isUnboxedTupleCon dc, VoidArg <- typeCgRep (idType bind2)
     = --trace "automagic mashing of case alts (# a, VoidArg #)" $
-     doCase d s p scrut bind1 [(DEFAULT, [], rhs)] True{-unboxed tuple-} 
+     doCase d s p scrut bind1 [(DEFAULT, [], rhs)] True{-unboxed tuple-}
  
  
-schemeE d s p (AnnCase scrut bndr _ [(DataAlt dc, [bind1], rhs)])
+schemeE d s p (AnnCase scrut _ _ [(DataAlt dc, [bind1], rhs)])
     | isUnboxedTupleCon dc
     | isUnboxedTupleCon dc
-       -- Similarly, convert
-       --      case .... of x { (# a #) -> ... }
-       -- to
-       --      case .... of a { DEFAULT -> ... }
+        -- Similarly, convert
+        --      case .... of x { (# a #) -> ... }
+        -- to
+        --      case .... of a { DEFAULT -> ... }
     = --trace "automagic mashing of case alts (# a #)"  $
     = --trace "automagic mashing of case alts (# a #)"  $
-     doCase d s p scrut bind1 [(DEFAULT, [], rhs)] True{-unboxed tuple-} 
+     doCase d s p scrut bind1 [(DEFAULT, [], rhs)] True{-unboxed tuple-}
  
  schemeE d s p (AnnCase scrut bndr _ alts)
  
  schemeE d s p (AnnCase scrut bndr _ alts)
-   = doCase d s p scrut bndr alts False{-not an unboxed tuple-} 
-
-schemeE d s p (AnnNote note (_, body))
-   = schemeE d s p body
-
-schemeE d s p (AnnCast (_, body) _)
-   = schemeE d s p body
+   = doCase d s p scrut bndr alts False{-not an unboxed tuple-}
  
  
-schemeE d s p other
-   = pprPanic "ByteCodeGen.schemeE: unhandled case" 
-               (pprCoreExpr (deAnnotate' other))
+schemeE _ _ _ expr
+   = pprPanic "ByteCodeGen.schemeE: unhandled case"
+               (pprCoreExpr (deAnnotate' expr))
  
  
-{- 
+{-
     Ticked Expressions
     ------------------
     Ticked Expressions
     ------------------
-  
+
     A ticked expression looks like this:
  
        case tick<n> var1 ... varN of DEFAULT -> e
     A ticked expression looks like this:
  
        case tick<n> var1 ... varN of DEFAULT -> e
@@ -526,22 +530,19 @@ schemeE d s p other
  
    otherwise we return Nothing.
  
  
    otherwise we return Nothing.
  
-  The idea is that the "case tick<n> ..." is really just an annotation on 
+  The idea is that the "case tick<n> ..." is really just an annotation on
    the code. When we find such a thing, we pull out the useful information,
    and then compile the code as if it was just the expression "e".
  
  -}
  
    the code. When we find such a thing, we pull out the useful information,
    and then compile the code as if it was just the expression "e".
  
  -}
  
-isTickedExp :: AnnExpr Id a -> Maybe (TickInfo, AnnExpr Id a)
-isTickedExp (annot, expr) = isTickedExp' expr 
-
  isTickedExp' :: AnnExpr' Id a -> Maybe (TickInfo, AnnExpr Id a)
  isTickedExp' (AnnCase scrut _bndr _type alts)
     | Just tickInfo <- isTickedScrut scrut,
  isTickedExp' :: AnnExpr' Id a -> Maybe (TickInfo, AnnExpr Id a)
  isTickedExp' (AnnCase scrut _bndr _type alts)
     | Just tickInfo <- isTickedScrut scrut,
-     [(DEFAULT, _bndr, rhs)] <- alts 
+     [(DEFAULT, _bndr, rhs)] <- alts
       = Just (tickInfo, rhs)
     where
       = Just (tickInfo, rhs)
     where
-   isTickedScrut :: (AnnExpr Id a) -> Maybe TickInfo 
+   isTickedScrut :: (AnnExpr Id a) -> Maybe TickInfo
     isTickedScrut expr
        | Var id <- f,
          Just (TickBox modName tickNumber) <- isTickBoxOp_maybe id
     isTickedScrut expr
        | Var id <- f,
          Just (TickBox modName tickNumber) <- isTickBoxOp_maybe id
@@ -553,12 +554,12 @@ isTickedExp' (AnnCase scrut _bndr _type alts)
        where
        (f, args) = collectArgs $ deAnnotate expr
        idsOfArgs :: [Expr Id] -> [Id]
        where
        (f, args) = collectArgs $ deAnnotate expr
        idsOfArgs :: [Expr Id] -> [Id]
-      idsOfArgs = catMaybes . map exprId 
+      idsOfArgs = catMaybes . map exprId
        exprId :: Expr Id -> Maybe Id
        exprId (Var id) = Just id
        exprId :: Expr Id -> Maybe Id
        exprId (Var id) = Just id
-      exprId other    = Nothing
+      exprId _        = Nothing
  
  
-isTickedExp' other = Nothing
+isTickedExp' _ = Nothing
  
  -- Compile code to do a tail call.  Specifically, push the fn,
  -- slide the on-stack app back down to the sequel depth,
  
  -- Compile code to do a tail call.  Specifically, push the fn,
  -- slide the on-stack app back down to the sequel depth,
@@ -577,16 +578,16 @@ isTickedExp' other = Nothing
  --     (# b #) and treat it as  b.
  --
  -- 3.  Application of a constructor, by defn saturated.
  --     (# b #) and treat it as  b.
  --
  -- 3.  Application of a constructor, by defn saturated.
---     Split the args into ptrs and non-ptrs, and push the nonptrs, 
+--     Split the args into ptrs and non-ptrs, and push the nonptrs,
  --     then the ptrs, and then do PACK and RETURN.
  --
  -- 4.  Otherwise, it must be a function call.  Push the args
  --     right to left, SLIDE and ENTER.
  
  --     then the ptrs, and then do PACK and RETURN.
  --
  -- 4.  Otherwise, it must be a function call.  Push the args
  --     right to left, SLIDE and ENTER.
  
-schemeT :: Int                 -- Stack depth
-        -> Sequel      -- Sequel depth
-        -> BCEnv       -- stack env
-        -> AnnExpr' Id VarSet 
+schemeT :: Word16       -- Stack depth
+        -> Sequel       -- Sequel depth
+        -> BCEnv        -- stack env
+        -> AnnExpr' Id VarSet
          -> BcM BCInstrList
  
  schemeT d s p app
          -> BcM BCInstrList
  
  schemeT d s p app
@@ -595,13 +596,13 @@ schemeT d s p app
  --   = panic "schemeT ?!?!"
  
  --   | trace ("\nschemeT\n" ++ showSDoc (pprCoreExpr (deAnnotate' app)) ++ "\n") False
  --   = panic "schemeT ?!?!"
  
  --   | trace ("\nschemeT\n" ++ showSDoc (pprCoreExpr (deAnnotate' app)) ++ "\n") False
---   = error "?!?!" 
+--   = error "?!?!"
  
     -- Case 0
     | Just (arg, constr_names) <- maybe_is_tagToEnum_call
     = do (push, arg_words) <- pushAtom d p arg
          tagToId_sequence <- implement_tagToId constr_names
  
     -- Case 0
     | Just (arg, constr_names) <- maybe_is_tagToEnum_call
     = do (push, arg_words) <- pushAtom d p arg
          tagToId_sequence <- implement_tagToId constr_names
-        return (push `appOL`  tagToId_sequence            
+        return (push `appOL`  tagToId_sequence
                         `appOL`  mkSLIDE 1 (d+arg_words-s)
                         `snocOL` ENTER)
  
                         `appOL`  mkSLIDE 1 (d+arg_words-s)
                         `snocOL` ENTER)
  
@@ -613,20 +614,20 @@ schemeT d s p app
     | Just con <- maybe_saturated_dcon,
       isUnboxedTupleCon con
     = case args_r_to_l of
     | Just con <- maybe_saturated_dcon,
       isUnboxedTupleCon con
     = case args_r_to_l of
-       [arg1,arg2] | isVoidArgAtom arg1 -> 
-                 unboxedTupleReturn d s p arg2
-       [arg1,arg2] | isVoidArgAtom arg2 -> 
-                 unboxedTupleReturn d s p arg1
-       _other -> unboxedTupleException
+        [arg1,arg2] | isVoidArgAtom arg1 ->
+                  unboxedTupleReturn d s p arg2
+        [arg1,arg2] | isVoidArgAtom arg2 ->
+                  unboxedTupleReturn d s p arg1
+        _other -> unboxedTupleException
  
     -- Case 3: Ordinary data constructor
     | Just con <- maybe_saturated_dcon
     = do alloc_con <- mkConAppCode d s p con args_r_to_l
  
     -- Case 3: Ordinary data constructor
     | Just con <- maybe_saturated_dcon
     = do alloc_con <- mkConAppCode d s p con args_r_to_l
-        return (alloc_con       `appOL` 
-                  mkSLIDE 1 (d - s) `snocOL`
-                  ENTER)
+        return (alloc_con         `appOL`
+                mkSLIDE 1 (d - s) `snocOL`
+                ENTER)
  
  
-   -- Case 4: Tail call of function 
+   -- Case 4: Tail call of function
     | otherwise
     = doTailCall d s p fn args_r_to_l
  
     | otherwise
     = doTailCall d s p fn args_r_to_l
  
@@ -634,55 +635,55 @@ schemeT d s p app
        -- Detect and extract relevant info for the tagToEnum kludge.
        maybe_is_tagToEnum_call
           = let extract_constr_Names ty
        -- Detect and extract relevant info for the tagToEnum kludge.
        maybe_is_tagToEnum_call
           = let extract_constr_Names ty
-                | Just (tyc, []) <- splitTyConApp_maybe (repType ty),
-                  isDataTyCon tyc
-                  = map (getName . dataConWorkId) (tyConDataCons tyc)
-                  -- NOTE: use the worker name, not the source name of
-                  -- the DataCon.  See DataCon.lhs for details.
-                | otherwise
-                  = panic "maybe_is_tagToEnum_call.extract_constr_Ids"
+                 | Just (tyc, _) <- splitTyConApp_maybe (repType ty),
+                   isDataTyCon tyc
+                   = map (getName . dataConWorkId) (tyConDataCons tyc)
+                   -- NOTE: use the worker name, not the source name of
+                   -- the DataCon.  See DataCon.lhs for details.
+                 | otherwise
+                   = pprPanic "maybe_is_tagToEnum_call.extract_constr_Ids" (ppr ty)
             in
             case app of
                (AnnApp (_, AnnApp (_, AnnVar v) (_, AnnType t)) arg)
                   -> case isPrimOpId_maybe v of
                         Just TagToEnumOp -> Just (snd arg, extract_constr_Names t)
             in
             case app of
                (AnnApp (_, AnnApp (_, AnnVar v) (_, AnnType t)) arg)
                   -> case isPrimOpId_maybe v of
                         Just TagToEnumOp -> Just (snd arg, extract_constr_Names t)
-                      other            -> Nothing
-              other -> Nothing
+                       _                -> Nothing
+              _ -> Nothing
  
  
-       -- Extract the args (R->L) and fn
-       -- The function will necessarily be a variable, 
-       -- because we are compiling a tail call
+        -- Extract the args (R->L) and fn
+        -- The function will necessarily be a variable,
+        -- because we are compiling a tail call
        (AnnVar fn, args_r_to_l) = splitApp app
  
        -- Only consider this to be a constructor application iff it is
        -- saturated.  Otherwise, we'll call the constructor wrapper.
        n_args = length args_r_to_l
        (AnnVar fn, args_r_to_l) = splitApp app
  
        -- Only consider this to be a constructor application iff it is
        -- saturated.  Otherwise, we'll call the constructor wrapper.
        n_args = length args_r_to_l
-      maybe_saturated_dcon  
-       = case isDataConWorkId_maybe fn of
-               Just con | dataConRepArity con == n_args -> Just con
-               _ -> Nothing
+      maybe_saturated_dcon
+        = case isDataConWorkId_maybe fn of
+                Just con | dataConRepArity con == n_args -> Just con
+                _ -> Nothing
  
  -- -----------------------------------------------------------------------------
  
  -- -----------------------------------------------------------------------------
--- Generate code to build a constructor application, 
+-- Generate code to build a constructor application,
  -- leaving it on top of the stack
  
  -- leaving it on top of the stack
  
-mkConAppCode :: Int -> Sequel -> BCEnv
-            -> DataCon                 -- The data constructor
-            -> [AnnExpr' Id VarSet]    -- Args, in *reverse* order
-            -> BcM BCInstrList
+mkConAppCode :: Word16 -> Sequel -> BCEnv
+             -> DataCon                 -- The data constructor
+             -> [AnnExpr' Id VarSet]    -- Args, in *reverse* order
+             -> BcM BCInstrList
  
  
-mkConAppCode orig_d s p con [] -- Nullary constructor
+mkConAppCode _ _ _ con []       -- Nullary constructor
    = ASSERT( isNullaryRepDataCon con )
      return (unitOL (PUSH_G (getName (dataConWorkId con))))
    = ASSERT( isNullaryRepDataCon con )
      return (unitOL (PUSH_G (getName (dataConWorkId con))))
-       -- Instead of doing a PACK, which would allocate a fresh
-       -- copy of this constructor, use the single shared version.
+        -- Instead of doing a PACK, which would allocate a fresh
+        -- copy of this constructor, use the single shared version.
  
  
-mkConAppCode orig_d s p con args_r_to_l 
+mkConAppCode orig_d _ p con args_r_to_l
    = ASSERT( dataConRepArity con == length args_r_to_l )
      do_pushery orig_d (non_ptr_args ++ ptr_args)
   where
    = ASSERT( dataConRepArity con == length args_r_to_l )
      do_pushery orig_d (non_ptr_args ++ ptr_args)
   where
-       -- The args are already in reverse order, which is the way PACK
-       -- expects them to be.  We must push the non-ptrs after the ptrs.
+        -- The args are already in reverse order, which is the way PACK
+        -- expects them to be.  We must push the non-ptrs after the ptrs.
        (ptr_args, non_ptr_args) = partition isPtrAtom args_r_to_l
  
        do_pushery d (arg:args)
        (ptr_args, non_ptr_args) = partition isPtrAtom args_r_to_l
  
        do_pushery d (arg:args)
@@ -691,8 +692,8 @@ mkConAppCode orig_d s p con args_r_to_l
                return (push `appOL` more_push_code)
        do_pushery d []
           = return (unitOL (PACK con n_arg_words))
                return (push `appOL` more_push_code)
        do_pushery d []
           = return (unitOL (PACK con n_arg_words))
-        where
-          n_arg_words = d - orig_d
+         where
+           n_arg_words = d - orig_d
  
  
  -- -----------------------------------------------------------------------------
  
  
  -- -----------------------------------------------------------------------------
@@ -703,46 +704,47 @@ mkConAppCode orig_d s p con args_r_to_l
  -- returned, even if it is a pointed type.  We always just return.
  
  unboxedTupleReturn
  -- returned, even if it is a pointed type.  We always just return.
  
  unboxedTupleReturn
-       :: Int -> Sequel -> BCEnv
-       -> AnnExpr' Id VarSet -> BcM BCInstrList
+        :: Word16 -> Sequel -> BCEnv
+        -> AnnExpr' Id VarSet -> BcM BCInstrList
  unboxedTupleReturn d s p arg = do
    (push, sz) <- pushAtom d p arg
  unboxedTupleReturn d s p arg = do
    (push, sz) <- pushAtom d p arg
-  return (push `appOL`
-           mkSLIDE sz (d-s) `snocOL`
-           RETURN_UBX (atomRep arg))
+  return (push                      `appOL`
+          mkSLIDE sz (d-s)          `snocOL`
+          RETURN_UBX (atomRep arg))
  
  -- -----------------------------------------------------------------------------
  -- Generate code for a tail-call
  
  doTailCall
  
  -- -----------------------------------------------------------------------------
  -- Generate code for a tail-call
  
  doTailCall
-       :: Int -> Sequel -> BCEnv
-       -> Id -> [AnnExpr' Id VarSet]
-       -> BcM BCInstrList
+        :: Word16 -> Sequel -> BCEnv
+        -> Id -> [AnnExpr' Id VarSet]
+        -> BcM BCInstrList
  doTailCall init_d s p fn args
    = do_pushes init_d args (map atomRep args)
    where
    do_pushes d [] reps = do
  doTailCall init_d s p fn args
    = do_pushes init_d args (map atomRep args)
    where
    do_pushes d [] reps = do
-       ASSERT( null reps ) return ()
+        ASSERT( null reps ) return ()
          (push_fn, sz) <- pushAtom d p (AnnVar fn)
          (push_fn, sz) <- pushAtom d p (AnnVar fn)
-       ASSERT( sz == 1 ) return ()
-       return (push_fn `appOL` (
-                 mkSLIDE ((d-init_d) + 1) (init_d - s) `appOL`
-                 unitOL ENTER))
+        ASSERT( sz == 1 ) return ()
+        return (push_fn `appOL` (
+                  mkSLIDE ((d-init_d) + 1) (init_d - s) `appOL`
+                  unitOL ENTER))
    do_pushes d args reps = do
        let (push_apply, n, rest_of_reps) = findPushSeq reps
    do_pushes d args reps = do
        let (push_apply, n, rest_of_reps) = findPushSeq reps
-         (these_args, rest_of_args) = splitAt n args
+          (these_args, rest_of_args) = splitAt n args
        (next_d, push_code) <- push_seq d these_args
        (next_d, push_code) <- push_seq d these_args
-      instrs <- do_pushes (next_d + 1) rest_of_args rest_of_reps 
-               --                ^^^ for the PUSH_APPLY_ instruction
+      instrs <- do_pushes (next_d + 1) rest_of_args rest_of_reps
+      --                          ^^^ for the PUSH_APPLY_ instruction
        return (push_code `appOL` (push_apply `consOL` instrs))
  
    push_seq d [] = return (d, nilOL)
    push_seq d (arg:args) = do
        return (push_code `appOL` (push_apply `consOL` instrs))
  
    push_seq d [] = return (d, nilOL)
    push_seq d (arg:args) = do
-    (push_code, sz) <- pushAtom d p arg 
+    (push_code, sz) <- pushAtom d p arg
      (final_d, more_push_code) <- push_seq (d+sz) args
      return (final_d, push_code `appOL` more_push_code)
  
  -- v. similar to CgStackery.findMatch, ToDo: merge
      (final_d, more_push_code) <- push_seq (d+sz) args
      return (final_d, push_code `appOL` more_push_code)
  
  -- v. similar to CgStackery.findMatch, ToDo: merge
+findPushSeq :: [CgRep] -> (BCInstr, Int, [CgRep])
  findPushSeq (PtrArg: PtrArg: PtrArg: PtrArg: PtrArg: PtrArg: rest)
    = (PUSH_APPLY_PPPPPP, 6, rest)
  findPushSeq (PtrArg: PtrArg: PtrArg: PtrArg: PtrArg: rest)
  findPushSeq (PtrArg: PtrArg: PtrArg: PtrArg: PtrArg: PtrArg: rest)
    = (PUSH_APPLY_PPPPPP, 6, rest)
  findPushSeq (PtrArg: PtrArg: PtrArg: PtrArg: PtrArg: rest)
@@ -771,11 +773,11 @@ findPushSeq _
  -- -----------------------------------------------------------------------------
  -- Case expressions
  
  -- -----------------------------------------------------------------------------
  -- Case expressions
  
-doCase  :: Int -> Sequel -> BCEnv
-       -> AnnExpr Id VarSet -> Id -> [AnnAlt Id VarSet]
-       -> Bool  -- True <=> is an unboxed tuple case, don't enter the result
-       -> BcM BCInstrList
-doCase d s p (_,scrut) bndr alts is_unboxed_tuple 
+doCase  :: Word16 -> Sequel -> BCEnv
+        -> AnnExpr Id VarSet -> Id -> [AnnAlt Id VarSet]
+        -> Bool  -- True <=> is an unboxed tuple case, don't enter the result
+        -> BcM BCInstrList
+doCase d s p (_,scrut) bndr alts is_unboxed_tuple
    = let
          -- Top of stack is the return itbl, as usual.
          -- underneath it is the pointer to the alt_code BCO.
    = let
          -- Top of stack is the return itbl, as usual.
          -- underneath it is the pointer to the alt_code BCO.
@@ -783,81 +785,84 @@ doCase d s p (_,scrut) bndr alts is_unboxed_tuple
          -- on top of the itbl.
          ret_frame_sizeW = 2
  
          -- on top of the itbl.
          ret_frame_sizeW = 2
  
-       -- An unlifted value gets an extra info table pushed on top
-       -- when it is returned.
-       unlifted_itbl_sizeW | isAlgCase = 0
-                           | otherwise = 1
+        -- An unlifted value gets an extra info table pushed on top
+        -- when it is returned.
+        unlifted_itbl_sizeW | isAlgCase = 0
+                            | otherwise = 1
  
  
-       -- depth of stack after the return value has been pushed
-       d_bndr = d + ret_frame_sizeW + idSizeW bndr
+        -- depth of stack after the return value has been pushed
+        d_bndr = d + ret_frame_sizeW + fromIntegral (idSizeW bndr)
  
  
-       -- depth of stack after the extra info table for an unboxed return
-       -- has been pushed, if any.  This is the stack depth at the
-       -- continuation.
+        -- depth of stack after the extra info table for an unboxed return
+        -- has been pushed, if any.  This is the stack depth at the
+        -- continuation.
          d_alts = d_bndr + unlifted_itbl_sizeW
  
          -- Env in which to compile the alts, not including
          -- any vars bound by the alts themselves
          d_alts = d_bndr + unlifted_itbl_sizeW
  
          -- Env in which to compile the alts, not including
          -- any vars bound by the alts themselves
-        p_alts = addToFM p bndr (d_bndr - 1)
+        p_alts = Map.insert bndr (d_bndr - 1) p
  
  
-       bndr_ty = idType bndr
+        bndr_ty = idType bndr
          isAlgCase = not (isUnLiftedType bndr_ty) && not is_unboxed_tuple
  
          -- given an alt, return a discr and code for it.
          isAlgCase = not (isUnLiftedType bndr_ty) && not is_unboxed_tuple
  
          -- given an alt, return a discr and code for it.
-       codeAlt alt@(DEFAULT, _, (_,rhs))
-          = do rhs_code <- schemeE d_alts s p_alts rhs
-               return (NoDiscr, rhs_code)
-
-        codeAlt alt@(discr, bndrs, (_,rhs))
-          -- primitive or nullary constructor alt: no need to UNPACK
-          | null real_bndrs = do
-               rhs_code <- schemeE d_alts s p_alts rhs
+        codeAlt (DEFAULT, _, (_,rhs))
+           = do rhs_code <- schemeE d_alts s p_alts rhs
+                return (NoDiscr, rhs_code)
+
+        codeAlt alt@(_, bndrs, (_,rhs))
+           -- primitive or nullary constructor alt: no need to UNPACK
+           | null real_bndrs = do
+                rhs_code <- schemeE d_alts s p_alts rhs
                  return (my_discr alt, rhs_code)
                  return (my_discr alt, rhs_code)
-          -- algebraic alt with some binders
-           | ASSERT(isAlgCase) otherwise =
+           -- algebraic alt with some binders
+           | otherwise =
               let
               let
-                (ptrs,nptrs) = partition (isFollowableArg.idCgRep) real_bndrs
-                ptr_sizes    = map idSizeW ptrs
-                nptrs_sizes  = map idSizeW nptrs
-                bind_sizes   = ptr_sizes ++ nptrs_sizes
-                size         = sum ptr_sizes + sum nptrs_sizes
-                -- the UNPACK instruction unpacks in reverse order...
-                p' = addListToFM p_alts 
-                       (zip (reverse (ptrs ++ nptrs))
-                         (mkStackOffsets d_alts (reverse bind_sizes)))
-            in do
-            rhs_code <- schemeE (d_alts+size) s p' rhs
+                 (ptrs,nptrs) = partition (isFollowableArg.idCgRep) real_bndrs
+                 ptr_sizes    = map (fromIntegral . idSizeW) ptrs
+                 nptrs_sizes  = map (fromIntegral . idSizeW) nptrs
+                 bind_sizes   = ptr_sizes ++ nptrs_sizes
+                 size         = sum ptr_sizes + sum nptrs_sizes
+                 -- the UNPACK instruction unpacks in reverse order...
+                 p' = Map.insertList
+                        (zip (reverse (ptrs ++ nptrs))
+                          (mkStackOffsets d_alts (reverse bind_sizes)))
+                        p_alts
+             in do
+             MASSERT(isAlgCase)
+             rhs_code <- schemeE (d_alts+size) s p' rhs
               return (my_discr alt, unitOL (UNPACK size) `appOL` rhs_code)
            where
               return (my_discr alt, unitOL (UNPACK size) `appOL` rhs_code)
            where
-            real_bndrs = filter (not.isTyVar) bndrs
+            real_bndrs = filterOut isTyVar bndrs
  
  
-        my_discr (DEFAULT, binds, rhs) = NoDiscr {-shouldn't really happen-}
-        my_discr (DataAlt dc, binds, rhs) 
+        my_discr (DEFAULT, _, _) = NoDiscr {-shouldn't really happen-}
+        my_discr (DataAlt dc, _, _)
             | isUnboxedTupleCon dc
             = unboxedTupleException
             | otherwise
             | isUnboxedTupleCon dc
             = unboxedTupleException
             | otherwise
-           = DiscrP (dataConTag dc - fIRST_TAG)
-        my_discr (LitAlt l, binds, rhs)
+           = DiscrP (fromIntegral (dataConTag dc - fIRST_TAG))
+        my_discr (LitAlt l, _, _)
             = case l of MachInt i     -> DiscrI (fromInteger i)
             = case l of MachInt i     -> DiscrI (fromInteger i)
+                       MachWord w    -> DiscrW (fromInteger w)
                         MachFloat r   -> DiscrF (fromRational r)
                         MachDouble r  -> DiscrD (fromRational r)
                         MachChar i    -> DiscrI (ord i)
                         _ -> pprPanic "schemeE(AnnCase).my_discr" (ppr l)
  
                         MachFloat r   -> DiscrF (fromRational r)
                         MachDouble r  -> DiscrD (fromRational r)
                         MachChar i    -> DiscrI (ord i)
                         _ -> pprPanic "schemeE(AnnCase).my_discr" (ppr l)
  
-        maybe_ncons 
+        maybe_ncons
             | not isAlgCase = Nothing
             | not isAlgCase = Nothing
-           | otherwise 
+           | otherwise
             = case [dc | (DataAlt dc, _, _) <- alts] of
                  []     -> Nothing
                  (dc:_) -> Just (tyConFamilySize (dataConTyCon dc))
  
             = case [dc | (DataAlt dc, _, _) <- alts] of
                  []     -> Nothing
                  (dc:_) -> Just (tyConFamilySize (dataConTyCon dc))
  
-       -- the bitmap is relative to stack depth d, i.e. before the
-       -- BCO, info table and return value are pushed on.
-       -- This bit of code is v. similar to buildLivenessMask in CgBindery,
-       -- except that here we build the bitmap from the known bindings of
-       -- things that are pointers, whereas in CgBindery the code builds the
-       -- bitmap from the free slots and unboxed bindings.
-       -- (ToDo: merge?)
+        -- the bitmap is relative to stack depth d, i.e. before the
+        -- BCO, info table and return value are pushed on.
+        -- This bit of code is v. similar to buildLivenessMask in CgBindery,
+        -- except that here we build the bitmap from the known bindings of
+        -- things that are pointers, whereas in CgBindery the code builds the
+        -- bitmap from the free slots and unboxed bindings.
+        -- (ToDo: merge?)
          --
          -- NOTE [7/12/2006] bug #1013, testcase ghci/should_run/ghci002.
          -- The bitmap must cover the portion of the stack up to the sequel only.
          --
          -- NOTE [7/12/2006] bug #1013, testcase ghci/should_run/ghci002.
          -- The bitmap must cover the portion of the stack up to the sequel only.
@@ -866,32 +871,34 @@ doCase d s p (_,scrut) bndr alts is_unboxed_tuple
          -- case-of-case expressions, which is the only time we can be compiling a
          -- case expression with s /= 0.
          bitmap_size = d-s
          -- case-of-case expressions, which is the only time we can be compiling a
          -- case expression with s /= 0.
          bitmap_size = d-s
-       bitmap = intsToReverseBitmap bitmap_size{-size-} 
-                        (sortLe (<=) (filter (< bitmap_size) rel_slots))
-         where
-         binds = fmToList p
-         rel_slots = concat (map spread binds)
-         spread (id, offset)
-               | isFollowableArg (idCgRep id) = [ rel_offset ]
-               | otherwise = []
-               where rel_offset = d - offset - 1
+        bitmap_size' :: Int
+        bitmap_size' = fromIntegral bitmap_size
+        bitmap = intsToReverseBitmap bitmap_size'{-size-}
+                        (sortLe (<=) (filter (< bitmap_size') rel_slots))
+          where
+          binds = Map.toList p
+          rel_slots = map fromIntegral $ concat (map spread binds)
+          spread (id, offset)
+                | isFollowableArg (idCgRep id) = [ rel_offset ]
+                | otherwise = []
+                where rel_offset = d - offset - 1
  
       in do
       alt_stuff <- mapM codeAlt alts
       alt_final <- mkMultiBranch maybe_ncons alt_stuff
  
  
       in do
       alt_stuff <- mapM codeAlt alts
       alt_final <- mkMultiBranch maybe_ncons alt_stuff
  
-     let 
+     let
           alt_bco_name = getName bndr
           alt_bco = mkProtoBCO alt_bco_name alt_final (Left alts)
           alt_bco_name = getName bndr
           alt_bco = mkProtoBCO alt_bco_name alt_final (Left alts)
-                       0{-no arity-} bitmap_size bitmap True{-is alts-}
+                       0{-no arity-} bitmap_size bitmap True{-is alts-}
       -- in
  --     trace ("case: bndr = " ++ showSDocDebug (ppr bndr) ++ "\ndepth = " ++ show d ++ "\nenv = \n" ++ showSDocDebug (ppBCEnv p) ++
       -- in
  --     trace ("case: bndr = " ++ showSDocDebug (ppr bndr) ++ "\ndepth = " ++ show d ++ "\nenv = \n" ++ showSDocDebug (ppBCEnv p) ++
---          "\n      bitmap = " ++ show bitmap) $ do
+--            "\n      bitmap = " ++ show bitmap) $ do
       scrut_code <- schemeE (d + ret_frame_sizeW) (d + ret_frame_sizeW) p scrut
       alt_bco' <- emitBc alt_bco
       let push_alts
       scrut_code <- schemeE (d + ret_frame_sizeW) (d + ret_frame_sizeW) p scrut
       alt_bco' <- emitBc alt_bco
       let push_alts
-           | isAlgCase = PUSH_ALTS alt_bco'
-           | otherwise = PUSH_ALTS_UNLIFTED alt_bco' (typeCgRep bndr_ty)
+            | isAlgCase = PUSH_ALTS alt_bco'
+            | otherwise = PUSH_ALTS_UNLIFTED alt_bco' (typeCgRep bndr_ty)
       return (push_alts `consOL` scrut_code)
  
  
       return (push_alts `consOL` scrut_code)
  
  
@@ -902,45 +909,46 @@ doCase d s p (_,scrut) bndr alts is_unboxed_tuple
  -- deferencing ForeignObj#s and adjusting addrs to point to
  -- payloads in Ptr/Byte arrays.  Then, generate the marshalling
  -- (machine) code for the ccall, and create bytecodes to call that and
  -- deferencing ForeignObj#s and adjusting addrs to point to
  -- payloads in Ptr/Byte arrays.  Then, generate the marshalling
  -- (machine) code for the ccall, and create bytecodes to call that and
--- then return in the right way.  
+-- then return in the right way.
  
  
-generateCCall :: Int -> Sequel                 -- stack and sequel depths
+generateCCall :: Word16 -> Sequel       -- stack and sequel depths
                -> BCEnv
                -> BCEnv
-              -> CCallSpec             -- where to call
-              -> Id                    -- of target, for type info
-              -> [AnnExpr' Id VarSet]  -- args (atoms)
+              -> CCallSpec              -- where to call
+              -> Id                     -- of target, for type info
+              -> [AnnExpr' Id VarSet]   -- args (atoms)
                -> BcM BCInstrList
  
                -> BcM BCInstrList
  
-generateCCall d0 s p ccall_spec@(CCallSpec target cconv safety) fn args_r_to_l
-   = let 
+generateCCall d0 s p (CCallSpec target cconv safety) fn args_r_to_l
+   = let
           -- useful constants
           -- useful constants
-         addr_sizeW = cgRepSizeW NonPtrArg
+         addr_sizeW :: Word16
+         addr_sizeW = fromIntegral (cgRepSizeW NonPtrArg)
  
           -- Get the args on the stack, with tags and suitably
           -- dereferenced for the CCall.  For each arg, return the
           -- depth to the first word of the bits for that arg, and the
           -- CgRep of what was actually pushed.
  
  
           -- Get the args on the stack, with tags and suitably
           -- dereferenced for the CCall.  For each arg, return the
           -- depth to the first word of the bits for that arg, and the
           -- CgRep of what was actually pushed.
  
-         pargs d [] = return []
-         pargs d (a:az) 
+         pargs _ [] = return []
+         pargs d (a:az)
              = let arg_ty = repType (exprType (deAnnotate' a))
  
                in case splitTyConApp_maybe arg_ty of
                      -- Don't push the FO; instead push the Addr# it
                      -- contains.
              = let arg_ty = repType (exprType (deAnnotate' a))
  
                in case splitTyConApp_maybe arg_ty of
                      -- Don't push the FO; instead push the Addr# it
                      -- contains.
-                   Just (t, _)
-                    | t == arrayPrimTyCon || t == mutableArrayPrimTyCon
+                    Just (t, _)
+                     | t == arrayPrimTyCon || t == mutableArrayPrimTyCon
                         -> do rest <- pargs (d + addr_sizeW) az
                         -> do rest <- pargs (d + addr_sizeW) az
-                             code <- parg_ArrayishRep arrPtrsHdrSize d p a
+                             code <- parg_ArrayishRep (fromIntegral arrPtrsHdrSize) d p a
                               return ((code,AddrRep):rest)
  
                               return ((code,AddrRep):rest)
  
-                    | t == byteArrayPrimTyCon || t == mutableByteArrayPrimTyCon
+                     | t == byteArrayPrimTyCon || t == mutableByteArrayPrimTyCon
                         -> do rest <- pargs (d + addr_sizeW) az
                         -> do rest <- pargs (d + addr_sizeW) az
-                             code <- parg_ArrayishRep arrWordsHdrSize d p a
+                             code <- parg_ArrayishRep (fromIntegral arrWordsHdrSize) d p a
                               return ((code,AddrRep):rest)
  
                      -- Default case: push taggedly, but otherwise intact.
                               return ((code,AddrRep):rest)
  
                      -- Default case: push taggedly, but otherwise intact.
-                    other
+                    _
                         -> do (code_a, sz_a) <- pushAtom d p a
                               rest <- pargs (d+sz_a) az
                               return ((code_a, atomPrimRep a) : rest)
                         -> do (code_a, sz_a) <- pushAtom d p a
                               rest <- pargs (d+sz_a) az
                               return ((code_a, atomPrimRep a) : rest)
@@ -948,6 +956,8 @@ generateCCall d0 s p ccall_spec@(CCallSpec target cconv safety) fn args_r_to_l
           -- Do magic for Ptr/Byte arrays.  Push a ptr to the array on
           -- the stack but then advance it over the headers, so as to
           -- point to the payload.
           -- Do magic for Ptr/Byte arrays.  Push a ptr to the array on
           -- the stack but then advance it over the headers, so as to
           -- point to the payload.
+         parg_ArrayishRep :: Word16 -> Word16 -> BCEnv -> AnnExpr' Id VarSet
+                          -> BcM BCInstrList
           parg_ArrayishRep hdrSize d p a
              = do (push_fo, _) <- pushAtom d p a
                   -- The ptr points at the header.  Advance it over the
           parg_ArrayishRep hdrSize d p a
              = do (push_fo, _) <- pushAtom d p a
                   -- The ptr points at the header.  Advance it over the
@@ -958,9 +968,10 @@ generateCCall d0 s p ccall_spec@(CCallSpec target cconv safety) fn args_r_to_l
       code_n_reps <- pargs d0 args_r_to_l
       let
           (pushs_arg, a_reps_pushed_r_to_l) = unzip code_n_reps
       code_n_reps <- pargs d0 args_r_to_l
       let
           (pushs_arg, a_reps_pushed_r_to_l) = unzip code_n_reps
+         a_reps_sizeW = fromIntegral (sum (map primRepSizeW a_reps_pushed_r_to_l))
  
           push_args    = concatOL pushs_arg
  
           push_args    = concatOL pushs_arg
-         d_after_args = d0 + sum (map primRepSizeW a_reps_pushed_r_to_l)
+         d_after_args = d0 + a_reps_sizeW
           a_reps_pushed_RAW
              | null a_reps_pushed_r_to_l || head a_reps_pushed_r_to_l /= VoidRep
              = panic "ByteCodeGen.generateCCall: missing or invalid World token?"
           a_reps_pushed_RAW
              | null a_reps_pushed_r_to_l || head a_reps_pushed_r_to_l /= VoidRep
              = panic "ByteCodeGen.generateCCall: missing or invalid World token?"
@@ -975,18 +986,18 @@ generateCCall d0 s p ccall_spec@(CCallSpec target cconv safety) fn args_r_to_l
           (returns_void, r_rep)
              = case maybe_getCCallReturnRep (idType fn) of
                   Nothing -> (True,  VoidRep)
           (returns_void, r_rep)
              = case maybe_getCCallReturnRep (idType fn) of
                   Nothing -> (True,  VoidRep)
-                 Just rr -> (False, rr) 
+                 Just rr -> (False, rr)
           {-
           {-
-         Because the Haskell stack grows down, the a_reps refer to 
+         Because the Haskell stack grows down, the a_reps refer to
           lowest to highest addresses in that order.  The args for the call
           are on the stack.  Now push an unboxed Addr# indicating
           lowest to highest addresses in that order.  The args for the call
           are on the stack.  Now push an unboxed Addr# indicating
-         the C function to call.  Then push a dummy placeholder for the 
-         result.  Finally, emit a CCALL insn with an offset pointing to the 
+         the C function to call.  Then push a dummy placeholder for the
+         result.  Finally, emit a CCALL insn with an offset pointing to the
           Addr# just pushed, and a literal field holding the mallocville
           address of the piece of marshalling code we generate.
           Addr# just pushed, and a literal field holding the mallocville
           address of the piece of marshalling code we generate.
-         So, just prior to the CCALL insn, the stack looks like this 
+         So, just prior to the CCALL insn, the stack looks like this
           (growing down, as usual):
           (growing down, as usual):
-                 
+
              <arg_n>
              ...
              <arg_1>
              <arg_n>
              ...
              <arg_1>
@@ -994,7 +1005,7 @@ generateCCall d0 s p ccall_spec@(CCallSpec target cconv safety) fn args_r_to_l
              <placeholder-for-result#> (must be an unboxed type)
  
           The interpreter then calls the marshall code mentioned
              <placeholder-for-result#> (must be an unboxed type)
  
           The interpreter then calls the marshall code mentioned
-         in the CCALL insn, passing it (& <placeholder-for-result#>), 
+         in the CCALL insn, passing it (& <placeholder-for-result#>),
           that is, the addr of the topmost word in the stack.
           When this returns, the placeholder will have been
           filled in.  The placeholder is slid down to the sequel
           that is, the addr of the topmost word in the stack.
           When this returns, the placeholder will have been
           filled in.  The placeholder is slid down to the sequel
@@ -1016,9 +1027,20 @@ generateCCall d0 s p ccall_spec@(CCallSpec target cconv safety) fn args_r_to_l
              = case target of
                   DynamicTarget
                      -> return (False, panic "ByteCodeGen.generateCCall(dyn)")
              = case target of
                   DynamicTarget
                      -> return (False, panic "ByteCodeGen.generateCCall(dyn)")
-                 StaticTarget target
-                    -> do res <- ioToBc (lookupStaticPtr target)
+
+                 StaticTarget target _
+                    -> do res <- ioToBc (lookupStaticPtr stdcall_adj_target)
                            return (True, res)
                            return (True, res)
+                   where
+                      stdcall_adj_target
+#ifdef mingw32_TARGET_OS
+                          | StdCallConv <- cconv
+                          = let size = fromIntegral a_reps_sizeW * wORD_SIZE in
+                            mkFastString (unpackFS target ++ '@':show size)
+#endif
+                          | otherwise
+                          = target
+
       -- in
       (is_static, static_target_addr) <- get_target_info
       let
       -- in
       (is_static, static_target_addr) <- get_target_info
       let
@@ -1026,7 +1048,7 @@ generateCCall d0 s p ccall_spec@(CCallSpec target cconv safety) fn args_r_to_l
           -- Get the arg reps, zapping the leading Addr# in the dynamic case
           a_reps --  | trace (showSDoc (ppr a_reps_pushed_RAW)) False = error "???"
                  | is_static = a_reps_pushed_RAW
           -- Get the arg reps, zapping the leading Addr# in the dynamic case
           a_reps --  | trace (showSDoc (ppr a_reps_pushed_RAW)) False = error "???"
                  | is_static = a_reps_pushed_RAW
-                | otherwise = if null a_reps_pushed_RAW 
+                | otherwise = if null a_reps_pushed_RAW
                                then panic "ByteCodeGen.generateCCall: dyn with no args"
                                else tail a_reps_pushed_RAW
  
                                then panic "ByteCodeGen.generateCCall: dyn with no args"
                                else tail a_reps_pushed_RAW
  
@@ -1035,30 +1057,25 @@ generateCCall d0 s p ccall_spec@(CCallSpec target cconv safety) fn args_r_to_l
              | is_static
              = (toOL [PUSH_UBX (Right static_target_addr) addr_sizeW],
                 d_after_args + addr_sizeW)
              | is_static
              = (toOL [PUSH_UBX (Right static_target_addr) addr_sizeW],
                 d_after_args + addr_sizeW)
-            | otherwise        -- is already on the stack
+            | otherwise -- is already on the stack
              = (nilOL, d_after_args)
  
           -- Push the return placeholder.  For a call returning nothing,
           -- this is a VoidArg (tag).
              = (nilOL, d_after_args)
  
           -- Push the return placeholder.  For a call returning nothing,
           -- this is a VoidArg (tag).
-         r_sizeW   = primRepSizeW r_rep
+         r_sizeW   = fromIntegral (primRepSizeW r_rep)
           d_after_r = d_after_Addr + r_sizeW
           r_lit     = mkDummyLiteral r_rep
           d_after_r = d_after_Addr + r_sizeW
           r_lit     = mkDummyLiteral r_rep
-         push_r    = (if   returns_void 
-                      then nilOL 
+         push_r    = (if   returns_void
+                      then nilOL
                        else unitOL (PUSH_UBX (Left r_lit) r_sizeW))
  
           -- generate the marshalling code we're going to call
                        else unitOL (PUSH_UBX (Left r_lit) r_sizeW))
  
           -- generate the marshalling code we're going to call
-         r_offW       = 0 
-         addr_offW    = r_sizeW
-         arg1_offW    = r_sizeW + addr_sizeW
-         args_offW    = map (arg1_offW +) 
-                            (init (scanl (+) 0 (map primRepSizeW a_reps)))
-
-        -- Offset of the next stack frame down the stack.  The CCALL
-        -- instruction needs to describe the chunk of stack containing
-        -- the ccall args to the GC, so it needs to know how large it
-        -- is.  See comment in Interpreter.c with the CCALL instruction.
-        stk_offset   = d_after_r - s
+
+         -- Offset of the next stack frame down the stack.  The CCALL
+         -- instruction needs to describe the chunk of stack containing
+         -- the ccall args to the GC, so it needs to know how large it
+         -- is.  See comment in Interpreter.c with the CCALL instruction.
+         stk_offset   = d_after_r - s
  
       -- in
       -- the only difference in libffi mode is that we prepare a cif
  
       -- in
       -- the only difference in libffi mode is that we prepare a cif
@@ -1070,7 +1087,8 @@ generateCCall d0 s p ccall_spec@(CCallSpec target cconv safety) fn args_r_to_l
       recordItblMallocBc (ItblPtr (castFunPtrToPtr addr_of_marshaller))
       let
           -- do the call
       recordItblMallocBc (ItblPtr (castFunPtrToPtr addr_of_marshaller))
       let
           -- do the call
-         do_call      = unitOL (CCALL stk_offset (castFunPtrToPtr addr_of_marshaller))
+         do_call      = unitOL (CCALL stk_offset (castFunPtrToPtr addr_of_marshaller)
+                                 (fromIntegral (fromEnum (playInterruptible safety))))
           -- slide and return
           wrapup       = mkSLIDE r_sizeW (d_after_r - r_sizeW - s)
                          `snocOL` RETURN_UBX (primRepToCgRep r_rep)
           -- slide and return
           wrapup       = mkSLIDE r_sizeW (d_after_r - r_sizeW - s)
                          `snocOL` RETURN_UBX (primRepToCgRep r_rep)
@@ -1096,7 +1114,7 @@ mkDummyLiteral pr
          _         -> panic "mkDummyLiteral"
  
  
          _         -> panic "mkDummyLiteral"
  
  
--- Convert (eg) 
+-- Convert (eg)
  --     GHC.Prim.Char# -> GHC.Prim.State# GHC.Prim.RealWorld
  --                   -> (# GHC.Prim.State# GHC.Prim.RealWorld, GHC.Prim.Int# #)
  --
  --     GHC.Prim.Char# -> GHC.Prim.State# GHC.Prim.RealWorld
  --                   -> (# GHC.Prim.State# GHC.Prim.RealWorld, GHC.Prim.Int# #)
  --
@@ -1112,10 +1130,10 @@ mkDummyLiteral pr
  
  maybe_getCCallReturnRep :: Type -> Maybe PrimRep
  maybe_getCCallReturnRep fn_ty
  
  maybe_getCCallReturnRep :: Type -> Maybe PrimRep
  maybe_getCCallReturnRep fn_ty
-   = let (a_tys, r_ty) = splitFunTys (dropForAlls fn_ty)
-         maybe_r_rep_to_go  
+   = let (_a_tys, r_ty) = splitFunTys (dropForAlls fn_ty)
+         maybe_r_rep_to_go
              = if isSingleton r_reps then Nothing else Just (r_reps !! 1)
              = if isSingleton r_reps then Nothing else Just (r_reps !! 1)
-         (r_tycon, r_reps) 
+         (r_tycon, r_reps)
              = case splitTyConApp_maybe (repType r_ty) of
                        (Just (tyc, tys)) -> (tyc, map typePrimRep tys)
                        Nothing -> blargh
              = case splitTyConApp_maybe (repType r_ty) of
                        (Just (tyc, tys)) -> (tyc, map typePrimRep tys)
                        Nothing -> blargh
@@ -1125,35 +1143,37 @@ maybe_getCCallReturnRep fn_ty
                && case maybe_r_rep_to_go of
                      Nothing    -> True
                      Just r_rep -> r_rep /= PtrRep
                && case maybe_r_rep_to_go of
                      Nothing    -> True
                      Just r_rep -> r_rep /= PtrRep
-                                  -- if it was, it would be impossible 
-                                  -- to create a valid return value 
+                                  -- if it was, it would be impossible
+                                  -- to create a valid return value
                                    -- placeholder on the stack
                                    -- placeholder on the stack
-         blargh = pprPanic "maybe_getCCallReturn: can't handle:" 
+
+         blargh :: a -- Used at more than one type
+         blargh = pprPanic "maybe_getCCallReturn: can't handle:"
                             (pprType fn_ty)
                             (pprType fn_ty)
-     in 
+     in
       --trace (showSDoc (ppr (a_reps, r_reps))) $
       if ok then maybe_r_rep_to_go else blargh
  
  -- Compile code which expects an unboxed Int on the top of stack,
       --trace (showSDoc (ppr (a_reps, r_reps))) $
       if ok then maybe_r_rep_to_go else blargh
  
  -- Compile code which expects an unboxed Int on the top of stack,
--- (call it i), and pushes the i'th closure in the supplied list 
+-- (call it i), and pushes the i'th closure in the supplied list
  -- as a consequence.
  implement_tagToId :: [Name] -> BcM BCInstrList
  implement_tagToId names
     = ASSERT( notNull names )
  -- as a consequence.
  implement_tagToId :: [Name] -> BcM BCInstrList
  implement_tagToId names
     = ASSERT( notNull names )
-     do labels <- getLabelsBc (length names)
+     do labels <- getLabelsBc (genericLength names)
          label_fail <- getLabelBc
          label_exit <- getLabelBc
          let infos = zip4 labels (tail labels ++ [label_fail])
                                  [0 ..] names
              steps = map (mkStep label_exit) infos
          return (concatOL steps
          label_fail <- getLabelBc
          label_exit <- getLabelBc
          let infos = zip4 labels (tail labels ++ [label_fail])
                                  [0 ..] names
              steps = map (mkStep label_exit) infos
          return (concatOL steps
-                  `appOL` 
+                  `appOL`
                    toOL [LABEL label_fail, CASEFAIL, LABEL label_exit])
       where
          mkStep l_exit (my_label, next_label, n, name_for_n)
                    toOL [LABEL label_fail, CASEFAIL, LABEL label_exit])
       where
          mkStep l_exit (my_label, next_label, n, name_for_n)
-           = toOL [LABEL my_label, 
-                   TESTEQ_I n next_label, 
-                   PUSH_G name_for_n, 
+           = toOL [LABEL my_label,
+                   TESTEQ_I n next_label,
+                   PUSH_G name_for_n,
                     JMP l_exit]
  
  
                     JMP l_exit]
  
  
@@ -1170,20 +1190,16 @@ implement_tagToId names
  -- to 5 and not to 4.  Stack locations are numbered from zero, so a
  -- depth 6 stack has valid words 0 .. 5.
  
  -- to 5 and not to 4.  Stack locations are numbered from zero, so a
  -- depth 6 stack has valid words 0 .. 5.
  
-pushAtom :: Int -> BCEnv -> AnnExpr' Id VarSet -> BcM (BCInstrList, Int)
-
-pushAtom d p (AnnApp f (_, AnnType _))
-   = pushAtom d p (snd f)
+pushAtom :: Word16 -> BCEnv -> AnnExpr' Id VarSet -> BcM (BCInstrList, Word16)
  
  
-pushAtom d p (AnnNote note e)
-   = pushAtom d p (snd e)
+pushAtom d p e
+   | Just e' <- bcView e
+   = pushAtom d p e'
  
  
-pushAtom d p (AnnLam x e) 
-   | isTyVar x 
-   = pushAtom d p (snd e)
+pushAtom _ _ (AnnCoercion {})  -- Coercions are zero-width things, 
+   = return (nilOL, 0)         -- treated just like a variable VoidArg
  
  pushAtom d p (AnnVar v)
  
  pushAtom d p (AnnVar v)
-
     | idCgRep v == VoidArg
     = return (nilOL, 0)
  
     | idCgRep v == VoidArg
     = return (nilOL, 0)
  
@@ -1193,58 +1209,62 @@ pushAtom d p (AnnVar v)
     | Just primop <- isPrimOpId_maybe v
     = return (unitOL (PUSH_PRIMOP primop), 1)
  
     | Just primop <- isPrimOpId_maybe v
     = return (unitOL (PUSH_PRIMOP primop), 1)
  
-   | Just d_v <- lookupBCEnv_maybe p v  -- v is a local variable
-   = return (toOL (nOfThem sz (PUSH_L (d-d_v+sz-2))), sz)
-        -- d - d_v                 the number of words between the TOS 
-        --                         and the 1st slot of the object
-        --
-        -- d - d_v - 1             the offset from the TOS of the 1st slot
-        --
-        -- d - d_v - 1 + sz - 1    the offset from the TOS of the last slot
-        --                         of the object.
-        --
-        -- Having found the last slot, we proceed to copy the right number of
-        -- slots on to the top of the stack.
+   | Just d_v <- lookupBCEnv_maybe v p  -- v is a local variable
+   = let l = d - d_v + sz - 2
+     in return (toOL (genericReplicate sz (PUSH_L l)), sz)
+         -- d - d_v                 the number of words between the TOS
+         --                         and the 1st slot of the object
+         --
+         -- d - d_v - 1             the offset from the TOS of the 1st slot
+         --
+         -- d - d_v - 1 + sz - 1    the offset from the TOS of the last slot
+         --                         of the object.
+         --
+         -- Having found the last slot, we proceed to copy the right number of
+         -- slots on to the top of the stack.
  
      | otherwise  -- v must be a global variable
  
      | otherwise  -- v must be a global variable
-    = ASSERT(sz == 1) 
+    = ASSERT(sz == 1)
        return (unitOL (PUSH_G (getName v)), sz)
  
      where
        return (unitOL (PUSH_G (getName v)), sz)
  
      where
-         sz = idSizeW v
+         sz :: Word16
+         sz = fromIntegral (idSizeW v)
  
  
  
  
-pushAtom d p (AnnLit lit)
+pushAtom _ _ (AnnLit lit)
     = case lit of
     = case lit of
-        MachLabel fs _ -> code NonPtrArg
-        MachWord w     -> code NonPtrArg
-        MachInt i      -> code PtrArg
-        MachFloat r    -> code FloatArg
-        MachDouble r   -> code DoubleArg
-        MachChar c     -> code NonPtrArg
-        MachStr s      -> pushStr s
+        MachLabel _ _ _ -> code NonPtrArg
+        MachWord _    -> code NonPtrArg
+        MachInt _     -> code PtrArg
+        MachFloat _   -> code FloatArg
+        MachDouble _  -> code DoubleArg
+        MachChar _    -> code NonPtrArg
+        MachNullAddr  -> code NonPtrArg
+        MachStr s     -> pushStr s
+        l             -> pprPanic "pushAtom" (ppr l)
       where
          code rep
       where
          code rep
-           = let size_host_words = cgRepSizeW rep
-             in  return (unitOL (PUSH_UBX (Left lit) size_host_words), 
+           = let size_host_words = fromIntegral (cgRepSizeW rep)
+             in  return (unitOL (PUSH_UBX (Left lit) size_host_words),
                             size_host_words)
  
                             size_host_words)
  
-        pushStr s 
+        pushStr s
             = let getMallocvilleAddr
                      = case s of
             = let getMallocvilleAddr
                      = case s of
-                         FastString _ n _ fp _ -> 
-                           -- we could grab the Ptr from the ForeignPtr,
-                           -- but then we have no way to control its lifetime.
-                           -- In reality it'll probably stay alive long enoungh
-                           -- by virtue of the global FastString table, but
-                           -- to be on the safe side we copy the string into
-                           -- a malloc'd area of memory.
+                         FastString _ n _ fp _ ->
+                            -- we could grab the Ptr from the ForeignPtr,
+                            -- but then we have no way to control its lifetime.
+                            -- In reality it'll probably stay alive long enoungh
+                            -- by virtue of the global FastString table, but
+                            -- to be on the safe side we copy the string into
+                            -- a malloc'd area of memory.
                                  do ptr <- ioToBc (mallocBytes (n+1))
                                     recordMallocBc ptr
                                     ioToBc (
                                        withForeignPtr fp $ \p -> do
                                  do ptr <- ioToBc (mallocBytes (n+1))
                                     recordMallocBc ptr
                                     ioToBc (
                                        withForeignPtr fp $ \p -> do
-                                        memcpy ptr p (fromIntegral n)
-                                        pokeByteOff ptr n (fromIntegral (ord '\0') :: Word8)
+                                         memcpy ptr p (fromIntegral n)
+                                         pokeByteOff ptr n (fromIntegral (ord '\0') :: Word8)
                                           return ptr
                                        )
               in do
                                           return ptr
                                        )
               in do
@@ -1252,12 +1272,9 @@ pushAtom d p (AnnLit lit)
                  -- Get the addr on the stack, untaggedly
                  return (unitOL (PUSH_UBX (Right addr) 1), 1)
  
                  -- Get the addr on the stack, untaggedly
                  return (unitOL (PUSH_UBX (Right addr) 1), 1)
  
-pushAtom d p (AnnCast e _)
-   = pushAtom d p (snd e)
-
-pushAtom d p other
-   = pprPanic "ByteCodeGen.pushAtom" 
-              (pprCoreExpr (deAnnotate (undefined, other)))
+pushAtom _ _ expr
+   = pprPanic "ByteCodeGen.pushAtom"
+              (pprCoreExpr (deAnnotate (undefined, expr)))
  
  foreign import ccall unsafe "memcpy"
   memcpy :: Ptr a -> Ptr b -> CSize -> IO ()
  
  foreign import ccall unsafe "memcpy"
   memcpy :: Ptr a -> Ptr b -> CSize -> IO ()
@@ -1268,29 +1285,29 @@ foreign import ccall unsafe "memcpy"
  -- of making a multiway branch using a switch tree.
  -- What a load of hassle!
  
  -- of making a multiway branch using a switch tree.
  -- What a load of hassle!
  
-mkMultiBranch :: Maybe Int     --  # datacons in tycon, if alg alt
-                               -- a hint; generates better code
-                               -- Nothing is always safe
-              -> [(Discr, BCInstrList)] 
+mkMultiBranch :: Maybe Int      -- # datacons in tycon, if alg alt
+                                -- a hint; generates better code
+                                -- Nothing is always safe
+              -> [(Discr, BCInstrList)]
                -> BcM BCInstrList
  mkMultiBranch maybe_ncons raw_ways
     = let d_way     = filter (isNoDiscr.fst) raw_ways
                -> BcM BCInstrList
  mkMultiBranch maybe_ncons raw_ways
     = let d_way     = filter (isNoDiscr.fst) raw_ways
-         notd_ways = sortLe 
+         notd_ways = sortLe
                          (\w1 w2 -> leAlt (fst w1) (fst w2))
                          (filter (not.isNoDiscr.fst) raw_ways)
  
           mkTree :: [(Discr, BCInstrList)] -> Discr -> Discr -> BcM BCInstrList
                          (\w1 w2 -> leAlt (fst w1) (fst w2))
                          (filter (not.isNoDiscr.fst) raw_ways)
  
           mkTree :: [(Discr, BCInstrList)] -> Discr -> Discr -> BcM BCInstrList
-         mkTree [] range_lo range_hi = return the_default
+         mkTree [] _range_lo _range_hi = return the_default
  
           mkTree [val] range_lo range_hi
  
           mkTree [val] range_lo range_hi
-            | range_lo `eqAlt` range_hi 
+            | range_lo `eqAlt` range_hi
              = return (snd val)
              | otherwise
              = do label_neq <- getLabelBc
              = return (snd val)
              | otherwise
              = do label_neq <- getLabelBc
-                 return (mkTestEQ (fst val) label_neq 
-                         `consOL` (snd val
-                         `appOL`   unitOL (LABEL label_neq)
-                          `appOL`   the_default))
+                 return (testEQ (fst val) label_neq
+                         `consOL` (snd val
+                         `appOL`   unitOL (LABEL label_neq)
+                         `appOL`   the_default))
  
           mkTree vals range_lo range_hi
              = let n = length vals `div` 2
  
           mkTree vals range_lo range_hi
              = let n = length vals `div` 2
@@ -1301,45 +1318,51 @@ mkMultiBranch maybe_ncons raw_ways
                label_geq <- getLabelBc
                code_lo <- mkTree vals_lo range_lo (dec v_mid)
                code_hi <- mkTree vals_hi v_mid range_hi
                label_geq <- getLabelBc
                code_lo <- mkTree vals_lo range_lo (dec v_mid)
                code_hi <- mkTree vals_hi v_mid range_hi
-              return (mkTestLT v_mid label_geq
-                        `consOL` (code_lo
-                       `appOL`   unitOL (LABEL label_geq)
-                       `appOL`   code_hi))
- 
-         the_default 
+              return (testLT v_mid label_geq
+                      `consOL` (code_lo
+                      `appOL`   unitOL (LABEL label_geq)
+                      `appOL`   code_hi))
+
+         the_default
              = case d_way of [] -> unitOL CASEFAIL
                              [(_, def)] -> def
              = case d_way of [] -> unitOL CASEFAIL
                              [(_, def)] -> def
+                            _ -> panic "mkMultiBranch/the_default"
+
+         testLT (DiscrI i) fail_label = TESTLT_I i fail_label
+         testLT (DiscrW i) fail_label = TESTLT_W i fail_label
+         testLT (DiscrF i) fail_label = TESTLT_F i fail_label
+         testLT (DiscrD i) fail_label = TESTLT_D i fail_label
+         testLT (DiscrP i) fail_label = TESTLT_P i fail_label
+         testLT NoDiscr    _          = panic "mkMultiBranch NoDiscr"
+
+         testEQ (DiscrI i) fail_label = TESTEQ_I i fail_label
+         testEQ (DiscrW i) fail_label = TESTEQ_W i fail_label
+         testEQ (DiscrF i) fail_label = TESTEQ_F i fail_label
+         testEQ (DiscrD i) fail_label = TESTEQ_D i fail_label
+         testEQ (DiscrP i) fail_label = TESTEQ_P i fail_label
+         testEQ NoDiscr    _          = panic "mkMultiBranch NoDiscr"
  
           -- None of these will be needed if there are no non-default alts
  
           -- None of these will be needed if there are no non-default alts
-         (mkTestLT, mkTestEQ, init_lo, init_hi)
+         (init_lo, init_hi)
              | null notd_ways
              = panic "mkMultiBranch: awesome foursome"
              | otherwise
              | null notd_ways
              = panic "mkMultiBranch: awesome foursome"
              | otherwise
-            = case fst (head notd_ways) of {
-              DiscrI _ -> ( \(DiscrI i) fail_label -> TESTLT_I i fail_label,
-                            \(DiscrI i) fail_label -> TESTEQ_I i fail_label,
-                            DiscrI minBound,
-                            DiscrI maxBound );
-              DiscrF _ -> ( \(DiscrF f) fail_label -> TESTLT_F f fail_label,
-                            \(DiscrF f) fail_label -> TESTEQ_F f fail_label,
-                            DiscrF minF,
-                            DiscrF maxF );
-              DiscrD _ -> ( \(DiscrD d) fail_label -> TESTLT_D d fail_label,
-                            \(DiscrD d) fail_label -> TESTEQ_D d fail_label,
-                            DiscrD minD,
-                            DiscrD maxD );
-              DiscrP _ -> ( \(DiscrP i) fail_label -> TESTLT_P i fail_label,
-                            \(DiscrP i) fail_label -> TESTEQ_P i fail_label,
-                            DiscrP algMinBound,
-                            DiscrP algMaxBound )
-              }
+            = case fst (head notd_ways) of
+                DiscrI _ -> ( DiscrI minBound,  DiscrI maxBound )
+                DiscrW _ -> ( DiscrW minBound,  DiscrW maxBound )
+                DiscrF _ -> ( DiscrF minF,      DiscrF maxF )
+                DiscrD _ -> ( DiscrD minD,      DiscrD maxD )
+                DiscrP _ -> ( DiscrP algMinBound, DiscrP algMaxBound )
+                NoDiscr -> panic "mkMultiBranch NoDiscr"
  
           (algMinBound, algMaxBound)
              = case maybe_ncons of
  
           (algMinBound, algMaxBound)
              = case maybe_ncons of
-                 Just n  -> (0, n - 1)
+                 -- XXX What happens when n == 0?
+                 Just n  -> (0, fromIntegral n - 1)
                   Nothing -> (minBound, maxBound)
  
           (DiscrI i1) `eqAlt` (DiscrI i2) = i1 == i2
                   Nothing -> (minBound, maxBound)
  
           (DiscrI i1) `eqAlt` (DiscrI i2) = i1 == i2
+         (DiscrW w1) `eqAlt` (DiscrW w2) = w1 == w2
           (DiscrF f1) `eqAlt` (DiscrF f2) = f1 == f2
           (DiscrD d1) `eqAlt` (DiscrD d2) = d1 == d2
           (DiscrP i1) `eqAlt` (DiscrP i2) = i1 == i2
           (DiscrF f1) `eqAlt` (DiscrF f2) = f1 == f2
           (DiscrD d1) `eqAlt` (DiscrD d2) = d1 == d2
           (DiscrP i1) `eqAlt` (DiscrP i2) = i1 == i2
@@ -1347,6 +1370,7 @@ mkMultiBranch maybe_ncons raw_ways
           _           `eqAlt` _           = False
  
           (DiscrI i1) `leAlt` (DiscrI i2) = i1 <= i2
           _           `eqAlt` _           = False
  
           (DiscrI i1) `leAlt` (DiscrI i2) = i1 <= i2
+         (DiscrW w1) `leAlt` (DiscrW w2) = w1 <= w2
           (DiscrF f1) `leAlt` (DiscrF f2) = f1 <= f2
           (DiscrD d1) `leAlt` (DiscrD d2) = d1 <= d2
           (DiscrP i1) `leAlt` (DiscrP i2) = i1 <= i2
           (DiscrF f1) `leAlt` (DiscrF f2) = f1 <= f2
           (DiscrD d1) `leAlt` (DiscrD d2) = d1 <= d2
           (DiscrP i1) `leAlt` (DiscrP i2) = i1 <= i2
@@ -1357,9 +1381,10 @@ mkMultiBranch maybe_ncons raw_ways
           isNoDiscr _       = False
  
           dec (DiscrI i) = DiscrI (i-1)
           isNoDiscr _       = False
  
           dec (DiscrI i) = DiscrI (i-1)
+         dec (DiscrW w) = DiscrW (w-1)
           dec (DiscrP i) = DiscrP (i-1)
           dec (DiscrP i) = DiscrP (i-1)
-         dec other      = other                -- not really right, but if you
-               -- do cases on floating values, you'll get what you deserve
+         dec other      = other         -- not really right, but if you
+                -- do cases on floating values, you'll get what you deserve
  
           -- same snotty comment applies to the following
           minF, maxF :: Float
  
           -- same snotty comment applies to the following
           minF, maxF :: Float
@@ -1376,68 +1401,75 @@ mkMultiBranch maybe_ncons raw_ways
  -- Supporting junk for the compilation schemes
  
  -- Describes case alts
  -- Supporting junk for the compilation schemes
  
  -- Describes case alts
-data Discr 
+data Discr
     = DiscrI Int
     = DiscrI Int
+   | DiscrW Word
     | DiscrF Float
     | DiscrD Double
     | DiscrF Float
     | DiscrD Double
-   | DiscrP Int
+   | DiscrP Word16
     | NoDiscr
  
  instance Outputable Discr where
     ppr (DiscrI i) = int i
     | NoDiscr
  
  instance Outputable Discr where
     ppr (DiscrI i) = int i
+   ppr (DiscrW w) = text (show w)
     ppr (DiscrF f) = text (show f)
     ppr (DiscrD d) = text (show d)
     ppr (DiscrF f) = text (show f)
     ppr (DiscrD d) = text (show d)
-   ppr (DiscrP i) = int i
+   ppr (DiscrP i) = ppr i
     ppr NoDiscr    = text "DEF"
  
  
     ppr NoDiscr    = text "DEF"
  
  
-lookupBCEnv_maybe :: BCEnv -> Id -> Maybe Int
-lookupBCEnv_maybe = lookupFM
+lookupBCEnv_maybe :: Id -> BCEnv -> Maybe Word16
+lookupBCEnv_maybe = Map.lookup
  
  idSizeW :: Id -> Int
  idSizeW id = cgRepSizeW (typeCgRep (idType id))
  
  -- See bug #1257
  unboxedTupleException :: a
  
  idSizeW :: Id -> Int
  idSizeW id = cgRepSizeW (typeCgRep (idType id))
  
  -- See bug #1257
  unboxedTupleException :: a
-unboxedTupleException 
-   = throwDyn 
-        (ProgramError 
+unboxedTupleException
+   = ghcError
+        (ProgramError
             ("Error: bytecode compiler can't handle unboxed tuples.\n"++
              "  Possibly due to foreign import/export decls in source.\n"++
              "  Workaround: use -fobject-code, or compile this module to .o separately."))
  
  
             ("Error: bytecode compiler can't handle unboxed tuples.\n"++
              "  Possibly due to foreign import/export decls in source.\n"++
              "  Workaround: use -fobject-code, or compile this module to .o separately."))
  
  
+mkSLIDE :: Word16 -> Word16 -> OrdList BCInstr
  mkSLIDE n d = if d == 0 then nilOL else unitOL (SLIDE n d)
  mkSLIDE n d = if d == 0 then nilOL else unitOL (SLIDE n d)
-bind x f    = f x
-
-splitApp :: AnnExpr' id ann -> (AnnExpr' id ann, [AnnExpr' id ann])
-       -- The arguments are returned in *right-to-left* order
-splitApp (AnnApp (_,f) (_,a))
-              | isTypeAtom a = splitApp f
-              | otherwise    = case splitApp f of 
-                                    (f', as) -> (f', a:as)
-splitApp (AnnNote n (_,e))    = splitApp e
-splitApp (AnnCast (_,e) _)    = splitApp e
-splitApp e                   = (e, [])
-
  
  
-isTypeAtom :: AnnExpr' id ann -> Bool
-isTypeAtom (AnnType _) = True
-isTypeAtom _           = False
-
-isVoidArgAtom :: AnnExpr' id ann -> Bool
-isVoidArgAtom (AnnVar v)        = typePrimRep (idType v) == VoidRep
-isVoidArgAtom (AnnNote n (_,e)) = isVoidArgAtom e
-isVoidArgAtom (AnnCast (_,e) _) = isVoidArgAtom e
-isVoidArgAtom _                = False
+splitApp :: AnnExpr' Var ann -> (AnnExpr' Var ann, [AnnExpr' Var ann])
+        -- The arguments are returned in *right-to-left* order
+splitApp e | Just e' <- bcView e = splitApp e'
+splitApp (AnnApp (_,f) (_,a))    = case splitApp f of
+                                      (f', as) -> (f', a:as)
+splitApp e                       = (e, [])
+
+
+bcView :: AnnExpr' Var ann -> Maybe (AnnExpr' Var ann)
+-- The "bytecode view" of a term discards
+--  a) type abstractions
+--  b) type applications
+--  c) casts
+--  d) notes
+-- Type lambdas *can* occur in random expressions,
+-- whereas value lambdas cannot; that is why they are nuked here
+bcView (AnnNote _ (_,e))            = Just e
+bcView (AnnCast (_,e) _)            = Just e
+bcView (AnnLam v (_,e)) | isTyVar v  = Just e
+bcView (AnnApp (_,e) (_, AnnType _)) = Just e
+bcView _                             = Nothing
+
+isVoidArgAtom :: AnnExpr' Var ann -> Bool
+isVoidArgAtom e | Just e' <- bcView e = isVoidArgAtom e'
+isVoidArgAtom (AnnVar v)              = typePrimRep (idType v) == VoidRep
+isVoidArgAtom (AnnCoercion {})        = True
+isVoidArgAtom _                      = False
  
  atomPrimRep :: AnnExpr' Id ann -> PrimRep
  
  atomPrimRep :: AnnExpr' Id ann -> PrimRep
-atomPrimRep (AnnVar v)    = typePrimRep (idType v)
-atomPrimRep (AnnLit l)    = typePrimRep (literalType l)
-atomPrimRep (AnnNote n b) = atomPrimRep (snd b)
-atomPrimRep (AnnApp f (_, AnnType _)) = atomPrimRep (snd f)
-atomPrimRep (AnnLam x e) | isTyVar x = atomPrimRep (snd e)
-atomPrimRep (AnnCast b _) = atomPrimRep (snd b)
+atomPrimRep e | Just e' <- bcView e = atomPrimRep e'
+atomPrimRep (AnnVar v)             = typePrimRep (idType v)
+atomPrimRep (AnnLit l)             = typePrimRep (literalType l)
+atomPrimRep (AnnCoercion {})        = VoidRep
  atomPrimRep other = pprPanic "atomPrimRep" (ppr (deAnnotate (undefined,other)))
  
  atomRep :: AnnExpr' Id ann -> CgRep
  atomPrimRep other = pprPanic "atomPrimRep" (ppr (deAnnotate (undefined,other)))
  
  atomRep :: AnnExpr' Id ann -> CgRep
@@ -1449,7 +1481,7 @@ isPtrAtom e = atomRep e == PtrArg
  -- Let szsw be the sizes in words of some items pushed onto the stack,
  -- which has initial depth d'.  Return the values which the stack environment
  -- should map these items to.
  -- Let szsw be the sizes in words of some items pushed onto the stack,
  -- which has initial depth d'.  Return the values which the stack environment
  -- should map these items to.
-mkStackOffsets :: Int -> [Int] -> [Int]
+mkStackOffsets :: Word16 -> [Word16] -> [Word16]
  mkStackOffsets original_depth szsw
     = map (subtract 1) (tail (scanl (+) original_depth szsw))
  
  mkStackOffsets original_depth szsw
     = map (subtract 1) (tail (scanl (+) original_depth szsw))
  
@@ -1458,38 +1490,38 @@ mkStackOffsets original_depth szsw
  
  type BcPtr = Either ItblPtr (Ptr ())
  
  
  type BcPtr = Either ItblPtr (Ptr ())
  
-data BcM_State 
-   = BcM_State { 
+data BcM_State
+   = BcM_State {
          uniqSupply :: UniqSupply,       -- for generating fresh variable names
          uniqSupply :: UniqSupply,       -- for generating fresh variable names
-       nextlabel :: Int,               -- for generating local labels
-       malloced  :: [BcPtr],           -- thunks malloced for current BCO
-                                       -- Should be free()d when it is GCd
-        breakArray :: BreakArray        -- array of breakpoint flags 
+        nextlabel :: Word16,            -- for generating local labels
+        malloced  :: [BcPtr],           -- thunks malloced for current BCO
+                                        -- Should be free()d when it is GCd
+        breakArray :: BreakArray        -- array of breakpoint flags
          }
  
  newtype BcM r = BcM (BcM_State -> IO (BcM_State, r))
  
  ioToBc :: IO a -> BcM a
          }
  
  newtype BcM r = BcM (BcM_State -> IO (BcM_State, r))
  
  ioToBc :: IO a -> BcM a
-ioToBc io = BcM $ \st -> do 
-  x <- io 
+ioToBc io = BcM $ \st -> do
+  x <- io
    return (st, x)
  
  runBc :: UniqSupply -> ModBreaks -> BcM r -> IO (BcM_State, r)
    return (st, x)
  
  runBc :: UniqSupply -> ModBreaks -> BcM r -> IO (BcM_State, r)
-runBc us modBreaks (BcM m) 
-   = m (BcM_State us 0 [] breakArray)   
+runBc us modBreaks (BcM m)
+   = m (BcM_State us 0 [] breakArray)
     where
     breakArray = modBreaks_flags modBreaks
  
  thenBc :: BcM a -> (a -> BcM b) -> BcM b
  thenBc (BcM expr) cont = BcM $ \st0 -> do
    (st1, q) <- expr st0
     where
     breakArray = modBreaks_flags modBreaks
  
  thenBc :: BcM a -> (a -> BcM b) -> BcM b
  thenBc (BcM expr) cont = BcM $ \st0 -> do
    (st1, q) <- expr st0
-  let BcM k = cont q 
+  let BcM k = cont q
    (st2, r) <- k st1
    return (st2, r)
  
  thenBc_ :: BcM a -> BcM b -> BcM b
  thenBc_ (BcM expr) (BcM cont) = BcM $ \st0 -> do
    (st2, r) <- k st1
    return (st2, r)
  
  thenBc_ :: BcM a -> BcM b -> BcM b
  thenBc_ (BcM expr) (BcM cont) = BcM $ \st0 -> do
-  (st1, q) <- expr st0
+  (st1, _) <- expr st0
    (st2, r) <- cont st1
    return (st2, r)
  
    (st2, r) <- cont st1
    return (st2, r)
  
@@ -1513,28 +1545,32 @@ recordItblMallocBc :: ItblPtr -> BcM ()
  recordItblMallocBc a
    = BcM $ \st -> return (st{malloced = Left a : malloced st}, ())
  
  recordItblMallocBc a
    = BcM $ \st -> return (st{malloced = Left a : malloced st}, ())
  
-getLabelBc :: BcM Int
+getLabelBc :: BcM Word16
  getLabelBc
  getLabelBc
-  = BcM $ \st -> return (st{nextlabel = 1 + nextlabel st}, nextlabel st)
+  = BcM $ \st -> do let nl = nextlabel st
+                    when (nl == maxBound) $
+                        panic "getLabelBc: Ran out of labels"
+                    return (st{nextlabel = nl + 1}, nl)
  
  
-getLabelsBc :: Int -> BcM [Int]
+getLabelsBc :: Word16 -> BcM [Word16]
  getLabelsBc n
  getLabelsBc n
-  = BcM $ \st -> let ctr = nextlabel st 
-                in return (st{nextlabel = ctr+n}, [ctr .. ctr+n-1])
+  = BcM $ \st -> let ctr = nextlabel st
+                 in return (st{nextlabel = ctr+n}, [ctr .. ctr+n-1])
  
  
-getBreakArray :: BcM BreakArray 
+getBreakArray :: BcM BreakArray
  getBreakArray = BcM $ \st -> return (st, breakArray st)
  
  newUnique :: BcM Unique
  newUnique = BcM $
  getBreakArray = BcM $ \st -> return (st, breakArray st)
  
  newUnique :: BcM Unique
  newUnique = BcM $
-   \st -> case splitUniqSupply (uniqSupply st) of
-             (us1, us2) -> let newState = st { uniqSupply = us2 } 
-                           in  return (newState, uniqFromSupply us1) 
+   \st -> case takeUniqFromSupply (uniqSupply st) of
+             (uniq, us) -> let newState = st { uniqSupply = us }
+                           in  return (newState, uniq)
  
  newId :: Type -> BcM Id
  
  newId :: Type -> BcM Id
-newId ty = do 
+newId ty = do
      uniq <- newUnique
      return $ mkSysLocal tickFS uniq ty
  
      uniq <- newUnique
      return $ mkSysLocal tickFS uniq ty
  
-tickFS = FSLIT("ticked")
+tickFS :: FastString
+tickFS = fsLit "ticked"
  \end{code}
  \end{code}