[project @ 2001-08-28 10:06:29 by simonpj]
[ghc-hetmet.git] / ghc / compiler / basicTypes / DataCon.lhs
index d916dcb..a925c1b 100644 (file)
@@ -8,46 +8,64 @@ module DataCon (
        DataCon,
        ConTag, fIRST_TAG,
        mkDataCon,
-       dataConType, dataConSig, dataConName, dataConTag,
-       dataConOrigArgTys, dataConArgTys, dataConRawArgTys, dataConTyCon,
-       dataConFieldLabels, dataConStrictMarks, dataConSourceArity,
-       dataConNumFields, dataConNumInstArgs, dataConId, dataConRepStrictness,
+       dataConRepType, dataConSig, dataConName, dataConTag, dataConTyCon,
+       dataConArgTys, dataConOrigArgTys, dataConInstOrigArgTys,
+       dataConRepArgTys, dataConTheta,
+       dataConFieldLabels, dataConStrictMarks,
+       dataConSourceArity, dataConRepArity,
+       dataConNumInstArgs, dataConId, dataConWrapId, dataConRepStrictness,
        isNullaryDataCon, isTupleCon, isUnboxedTupleCon,
-       isExistentialDataCon,
+       isExistentialDataCon, classDataCon,
 
-       StrictnessMark(..),     -- Representation visible to MkId only
-       markedStrict, notMarkedStrict, markedUnboxed, maybeMarkedUnboxed
+       splitProductType_maybe, splitProductType,
     ) where
 
 #include "HsVersions.h"
 
-import {-# SOURCE #-} Subst( substTy, mkTyVarSubst )
+import {-# SOURCE #-} Subst( substTyWith )
+import {-# SOURCE #-} PprType( pprType )
 
 import CmdLineOpts     ( opt_DictsStrict )
-import TysPrim
-import Type            ( Type, ThetaType, TauType,
-                         mkSigmaTy, mkFunTys, mkTyConApp, 
-                         mkTyVarTys, mkDictTy,
-                         splitAlgTyConApp_maybe
+import Type            ( Type, TauType, ThetaType, 
+                         mkForAllTys, mkFunTys, mkTyConApp,
+                         mkTyVarTys, splitTyConApp_maybe, repType, 
+                         mkPredTys, isStrictType
                        )
-import PprType
-import TyCon           ( TyCon, tyConDataCons, isDataTyCon,
-                         isTupleTyCon, isUnboxedTupleTyCon )
-import Class           ( classTyCon )
-import Name            ( Name, NamedThing(..), nameUnique, isLocallyDefinedName )
+import TyCon           ( TyCon, tyConDataCons, tyConDataConsIfAvailable, isProductTyCon,
+                         isTupleTyCon, isUnboxedTupleTyCon, isRecursiveTyCon )
+import Class           ( Class, classTyCon )
+import Name            ( Name, NamedThing(..), nameUnique )
 import Var             ( TyVar, Id )
 import FieldLabel      ( FieldLabel )
-import BasicTypes      ( Arity )
-import Demand          ( Demand, wwStrict, wwLazy )
+import BasicTypes      ( Arity, StrictnessMark(..) )
+import NewDemand       ( Demand, lazyDmd, seqDmd )
 import Outputable
 import Unique          ( Unique, Uniquable(..) )
 import CmdLineOpts     ( opt_UnboxStrictFields )
-import UniqSet
 import Maybe
-import Util            ( assoc )
+import ListSetOps      ( assoc )
+import Util            ( zipEqual, zipWithEqual )
 \end{code}
 
 
+Stuff about data constructors
+~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
+Every constructor, C, comes with a
+
+  *wrapper*, called C, whose type is exactly what it looks like
+       in the source program. It is an ordinary function,
+       and it gets a top-level binding like any other function
+
+  *worker*, called $wC, which is the actual data constructor.
+       Its type may be different to C, because:
+               - useless dict args are dropped
+               - strict args may be flattened
+       It does not have a binding.
+
+  The worker is very like a primop, in that it has no binding,
+
+
+
 %************************************************************************
 %*                                                                     *
 \subsection{Data constructors}
@@ -66,9 +84,9 @@ data DataCon
        --
        --      data Eq a => T a = forall b. Ord b => MkT a [b]
 
-       dcType   :: Type,       -- Type of the constructor 
+       dcRepType   :: Type,    -- Type of the constructor
                                --      forall ab . Ord b => a -> [b] -> MkT a
-                               -- (this is *not* of the constructor Id: 
+                               -- (this is *not* of the constructor Id:
                                --  see notes after this data type declaration)
 
        -- The next six fields express the type of the constructor, in pieces
@@ -82,40 +100,48 @@ data DataCon
        --      dcTyCon    = T
 
        dcTyVars :: [TyVar],            -- Type vars and context for the data type decl
+                                       -- These are ALWAYS THE SAME AS THE TYVARS
+                                       -- FOR THE PARENT TyCon.  We occasionally rely on
+                                       -- this just to avoid redundant instantiation
        dcTheta  ::  ThetaType,
 
-       dcExTyVars :: [TyVar],          -- Ditto for the context of the constructor, 
+       dcExTyVars :: [TyVar],          -- Ditto for the context of the constructor,
        dcExTheta  :: ThetaType,        -- the existentially quantified stuff
                                        
        dcOrigArgTys :: [Type],         -- Original argument types
                                        -- (before unboxing and flattening of
                                        --  strict fields)
-       dcRepArgTys :: [Type],          -- Constructor Argument types
-       dcTyCon  :: TyCon,              -- Result tycon 
 
-       -- Now the strictness annotations and field labels of the constructor
-       dcUserStricts :: [StrictnessMark], 
-               -- Strictness annotations, as placed on the data type defn,
-               -- in the same order as the argument types;
-               -- length = dataConNumFields dataCon
+       dcRepArgTys :: [Type],          -- Final, representation argument types, after unboxing and flattening,
+                                       -- and including existential dictionaries
+
+       dcRepStrictness :: [Demand],    -- One for each representation argument 
 
-       dcRealStricts :: [StrictnessMark],
-               -- Strictness annotations as deduced by the compiler.  May
-               -- include some MarkedUnboxed fields that are MarkedStrict
-               -- in dcUserStricts.
-               -- length = dataConNumFields dataCon
+       dcTyCon  :: TyCon,              -- Result tycon
+
+       -- Now the strictness annotations and field labels of the constructor
+       dcStrictMarks :: [StrictnessMark],
+               -- Strictness annotations as deduced by the compiler.  
+               -- Has no MarkedUserStrict; they have been changed to MarkedStrict
+               -- or MarkedUnboxed by the compiler.
+               -- *Includes the existential dictionaries*
+               -- length = length dcExTheta + dataConSourceArity dataCon
 
        dcFields  :: [FieldLabel],
                -- Field labels for this constructor, in the
                -- same order as the argument types; 
                -- length = 0 (if not a record) or dataConSourceArity.
 
-       -- Finally, the curried function that corresponds to the constructor
-       --      mkT :: forall a b. (Eq a, Ord b) => a -> [b] -> T a
-       --      mkT = /\ab. \deq dord p qs. Con MkT [a, b, dord, p, qs]
-       -- This unfolding is built in MkId.mkDataConId
+       -- Finally, the curried worker function that corresponds to the constructor
+       -- It doesn't have an unfolding; the code generator saturates these Ids
+       -- and allocates a real constructor when it finds one.
+       --
+       -- An entirely separate wrapper function is built in TcTyDecls
+
+       dcId :: Id,             -- The corresponding worker Id
+                               -- Takes dcRepArgTys as its arguments
 
-       dcId :: Id                      -- The corresponding Id
+       dcWrapId :: Id          -- The wrapper Id
   }
 
 type ConTag = Int
@@ -124,7 +150,7 @@ fIRST_TAG :: ConTag
 fIRST_TAG =  1 -- Tags allocated from here for real constructors
 \end{code}
 
-The dcType field contains the type of the representation of a contructor
+The dcRepType field contains the type of the representation of a contructor
 This may differ from the type of the contructor *Id* (built
 by MkId.mkDataConId) for two reasons:
        a) the constructor Id may be overloaded, but the dictionary isn't stored
@@ -143,26 +169,6 @@ Actually, the unboxed part isn't implemented yet!
 
 %************************************************************************
 %*                                                                     *
-\subsection{Strictness indication}
-%*                                                                     *
-%************************************************************************
-
-\begin{code}
-data StrictnessMark = MarkedStrict
-                   | MarkedUnboxed DataCon [Type]
-                   | NotMarkedStrict
-
-markedStrict    = MarkedStrict
-notMarkedStrict = NotMarkedStrict
-markedUnboxed   = MarkedUnboxed (panic "markedUnboxed1") (panic "markedUnboxed2")
-
-maybeMarkedUnboxed (MarkedUnboxed dc tys) = Just (dc,tys)
-maybeMarkedUnboxed other                 = Nothing
-\end{code}
-
-
-%************************************************************************
-%*                                                                     *
 \subsection{Instances}
 %*                                                                     *
 %************************************************************************
@@ -205,11 +211,13 @@ mkDataCon :: Name
          -> [TyVar] -> ThetaType
          -> [TyVar] -> ThetaType
          -> [TauType] -> TyCon
-         -> Id
+         -> Id -> Id
          -> DataCon
   -- Can get the tag from the TyCon
 
-mkDataCon name arg_stricts fields tyvars theta ex_tyvars ex_theta orig_arg_tys tycon id
+mkDataCon name arg_stricts fields
+         tyvars theta ex_tyvars ex_theta orig_arg_tys tycon
+         work_id wrap_id
   = ASSERT(length arg_stricts == length orig_arg_tys)
        -- The 'stricts' passed to mkDataCon are simply those for the
        -- source-language arguments.  We add extra ones for the
@@ -217,104 +225,40 @@ mkDataCon name arg_stricts fields tyvars theta ex_tyvars ex_theta orig_arg_tys t
     con
   where
     con = MkData {dcName = name, dcUnique = nameUnique name,
-                 dcTyVars = tyvars, dcTheta = theta, 
-                 dcOrigArgTys = orig_arg_tys, 
+                 dcTyVars = tyvars, dcTheta = theta,
+                 dcOrigArgTys = orig_arg_tys,
                  dcRepArgTys = rep_arg_tys,
                  dcExTyVars = ex_tyvars, dcExTheta = ex_theta,
-                 dcRealStricts = all_stricts, dcUserStricts = user_stricts,
-                 dcFields = fields, dcTag = tag, dcTyCon = tycon, dcType = ty,
-                 dcId = id}
-
-    (real_arg_stricts, strict_arg_tyss) 
-       = unzip (zipWith (unbox_strict_arg_ty tycon) arg_stricts orig_arg_tys)
-    rep_arg_tys = concat strict_arg_tyss
-       
-    ex_dict_stricts = map mk_dict_strict_mark ex_theta
-       -- Add a strictness flag for the existential dictionary arguments
-    all_stricts     = ex_dict_stricts ++ real_arg_stricts
-    user_stricts    = ex_dict_stricts ++ arg_stricts
+                 dcStrictMarks = real_stricts, dcRepStrictness = rep_arg_demands,
+                 dcFields = fields, dcTag = tag, dcTyCon = tycon, dcRepType = ty,
+                 dcId = work_id, dcWrapId = wrap_id}
+
+       -- Strictness marks for source-args
+       --      *after unboxing choices*, 
+       -- but  *including existential dictionaries*
+    ex_dict_tys  = mkPredTys ex_theta
+    real_stricts = (map mk_dict_strict_mark ex_dict_tys) ++
+                  zipWithEqual "mkDataCon1" (chooseBoxingStrategy tycon) 
+                               orig_arg_tys arg_stricts 
+
+       -- Representation arguments and demands
+    (rep_arg_demands, rep_arg_tys) 
+       = unzip $ concat $ 
+         zipWithEqual "mkDataCon2" unbox_strict_arg_ty 
+                      real_stricts 
+                      (ex_dict_tys ++ orig_arg_tys)
 
     tag = assoc "mkDataCon" (tyConDataCons tycon `zip` [fIRST_TAG..]) con
-    ty  = mkSigmaTy (tyvars ++ ex_tyvars) 
-                   ex_theta
-                   (mkFunTys rep_arg_tys 
-                       (mkTyConApp tycon (mkTyVarTys tyvars)))
+    ty  = mkForAllTys (tyvars ++ ex_tyvars)
+                     (mkFunTys rep_arg_tys result_ty)
+               -- NB: the existential dict args are already in rep_arg_tys
 
-mk_dict_strict_mark (clas,tys)
-  | opt_DictsStrict &&
-       -- Don't mark newtype things as strict!
-    isDataTyCon (classTyCon clas) = MarkedStrict
-  | otherwise                    = NotMarkedStrict
+    result_ty = mkTyConApp tycon (mkTyVarTys tyvars)
 
--- We attempt to unbox/unpack a strict field when either:
---   (i)  The tycon is imported, and the field is marked '! !', or
---   (ii) The tycon is defined in this module, the field is marked '!', 
---       and the -funbox-strict-fields flag is on.
---
--- This ensures that if we compile some modules with -funbox-strict-fields and
--- some without, the compiler doesn't get confused about the constructor
--- representations.
-
-unbox_strict_arg_ty :: TyCon -> StrictnessMark -> Type -> (StrictnessMark, [Type])
-unbox_strict_arg_ty tycon NotMarkedStrict ty 
-  = (NotMarkedStrict, [ty])
-unbox_strict_arg_ty tycon MarkedStrict ty 
-  | not opt_UnboxStrictFields
-  || not (isLocallyDefinedName (getName tycon)) = (MarkedStrict, [ty])
-unbox_strict_arg_ty tycon marked_unboxed ty
-  -- MarkedUnboxed || (MarkedStrict && opt_UnboxStrictFields && not imported)
-  = case splitAlgTyConApp_maybe ty of
-       Just (tycon,_,[])
-          -> panic (showSDoc (hcat [
-                       text "unbox_strict_arg_ty: constructors for ",
-                       ppr tycon,
-                       text " not available."
-                    ]))
-       Just (tycon,ty_args,[con]) 
-          -> case maybe_unpack_fields emptyUniqSet 
-                    (zip (dataConOrigArgTys con ty_args) 
-                         (dcUserStricts con))
-             of 
-                Nothing  -> (MarkedStrict, [ty])
-                Just tys -> (MarkedUnboxed con tys, tys)
-       _ -> (MarkedStrict, [ty])
-
--- bail out if we encounter the same tycon twice.  This avoids problems like
---
---   data A = !B
---   data B = !A
---
--- where no useful unpacking can be done.
-
-maybe_unpack_field :: UniqSet TyCon -> Type -> StrictnessMark -> Maybe [Type]
-maybe_unpack_field set ty NotMarkedStrict
-  = Just [ty]
-maybe_unpack_field set ty MarkedStrict | not opt_UnboxStrictFields
-  = Just [ty]
-maybe_unpack_field set ty strict
-  = case splitAlgTyConApp_maybe ty of
-       Just (tycon,ty_args,[con])
-               -- loop breaker
-          | tycon `elementOfUniqSet` set -> Nothing
-               -- don't unpack constructors with existential tyvars
-          | not (null ex_tyvars) -> Nothing
-               -- ok, let's do it
-          | otherwise ->
-               let set' = addOneToUniqSet set tycon in
-               maybe_unpack_fields set' 
-                   (zip (dataConOrigArgTys con ty_args)
-                        (dcUserStricts con))
-          where (_, _, ex_tyvars, _, _, _) = dataConSig con
-       _ -> Just [ty]
-
-maybe_unpack_fields :: UniqSet TyCon -> [(Type,StrictnessMark)] -> Maybe [Type]
-maybe_unpack_fields set tys
-  | all isJust unpacked_fields = Just (concat (catMaybes unpacked_fields))
-  | otherwise = Nothing
-  where unpacked_fields = map (\(ty,str) -> maybe_unpack_field set ty str) tys
+mk_dict_strict_mark ty | isStrictType ty = MarkedStrict
+                      | otherwise       = NotMarkedStrict
 \end{code}
 
-
 \begin{code}
 dataConName :: DataCon -> Name
 dataConName = dcName
@@ -325,39 +269,45 @@ dataConTag  = dcTag
 dataConTyCon :: DataCon -> TyCon
 dataConTyCon = dcTyCon
 
-dataConType :: DataCon -> Type
-dataConType = dcType
+dataConRepType :: DataCon -> Type
+dataConRepType = dcRepType
 
 dataConId :: DataCon -> Id
 dataConId = dcId
 
+dataConWrapId :: DataCon -> Id
+dataConWrapId = dcWrapId
 
 dataConFieldLabels :: DataCon -> [FieldLabel]
 dataConFieldLabels = dcFields
 
 dataConStrictMarks :: DataCon -> [StrictnessMark]
-dataConStrictMarks = dcRealStricts
+dataConStrictMarks = dcStrictMarks
 
-dataConRawArgTys :: DataCon -> [TauType] -- a function of convenience
-dataConRawArgTys = dcRepArgTys
+-- Number of type-instantiation arguments
+-- All the remaining arguments of the DataCon are (notionally)
+-- stored in the DataCon, and are matched in a case expression
+dataConNumInstArgs (MkData {dcTyVars = tyvars}) = length tyvars
 
 dataConSourceArity :: DataCon -> Arity
        -- Source-level arity of the data constructor
 dataConSourceArity dc = length (dcOrigArgTys dc)
 
+-- dataConRepArity gives the number of actual fields in the
+-- {\em representation} of the data constructor.  This may be more than appear
+-- in the source code; the extra ones are the existentially quantified
+-- dictionaries
+dataConRepArity (MkData {dcRepArgTys = arg_tys}) = length arg_tys
+
+isNullaryDataCon con  = dataConRepArity con == 0
+
 dataConRepStrictness :: DataCon -> [Demand]
-       -- Give the demands on the arguments of a 
+       -- Give the demands on the arguments of a
        -- Core constructor application (Con dc args)
-dataConRepStrictness dc
-  = go (dcRealStricts dc) 
-  where
-    go []                        = []
-    go (MarkedStrict        : ss) = wwStrict : go ss
-    go (NotMarkedStrict     : ss) = wwLazy   : go ss
-    go (MarkedUnboxed con _ : ss) = go (dcRealStricts con ++ ss)
+dataConRepStrictness dc = dcRepStrictness dc
 
-dataConSig :: DataCon -> ([TyVar], ThetaType, 
-                         [TyVar], ThetaType, 
+dataConSig :: DataCon -> ([TyVar], ThetaType,
+                         [TyVar], ThetaType,
                          [TauType], TyCon)
 
 dataConSig (MkData {dcTyVars = tyvars, dcTheta = theta,
@@ -365,41 +315,47 @@ dataConSig (MkData {dcTyVars = tyvars, dcTheta = theta,
                     dcOrigArgTys = arg_tys, dcTyCon = tycon})
   = (tyvars, theta, ex_tyvars, ex_theta, arg_tys, tycon)
 
-dataConArgTys, dataConOrigArgTys :: DataCon 
+dataConArgTys :: DataCon
              -> [Type]         -- Instantiated at these types
                                -- NB: these INCLUDE the existentially quantified arg types
              -> [Type]         -- Needs arguments of these types
                                -- NB: these INCLUDE the existentially quantified dict args
                                --     but EXCLUDE the data-decl context which is discarded
+                               -- It's all post-flattening etc; this is a representation type
+
+dataConArgTys (MkData {dcRepArgTys = arg_tys, dcTyVars = tyvars,
+                      dcExTyVars = ex_tyvars}) inst_tys
+ = map (substTyWith (tyvars ++ ex_tyvars) inst_tys) arg_tys
 
-dataConArgTys (MkData {dcRepArgTys = arg_tys, dcTyVars = tyvars, 
-                      dcExTyVars = ex_tyvars, dcExTheta = ex_theta}) inst_tys
- = map (substTy (mkTyVarSubst (tyvars ++ ex_tyvars) inst_tys)) 
-       ([mkDictTy cls tys | (cls,tys) <- ex_theta] ++ arg_tys)
+dataConTheta :: DataCon -> ThetaType
+dataConTheta dc = dcTheta dc
 
-dataConOrigArgTys (MkData {dcOrigArgTys = arg_tys, dcTyVars = tyvars, 
-                      dcExTyVars = ex_tyvars, dcExTheta = ex_theta}) inst_tys
- = map (substTy (mkTyVarSubst (tyvars ++ ex_tyvars) inst_tys)) 
-       ([mkDictTy cls tys | (cls,tys) <- ex_theta] ++ arg_tys)
+-- And the same deal for the original arg tys:
+
+dataConInstOrigArgTys :: DataCon -> [Type] -> [Type]
+dataConInstOrigArgTys (MkData {dcOrigArgTys = arg_tys, dcTyVars = tyvars,
+                      dcExTyVars = ex_tyvars}) inst_tys
+ = map (substTyWith (tyvars ++ ex_tyvars) inst_tys) arg_tys
 \end{code}
 
-dataConNumFields gives the number of actual fields in the
-{\em representation} of the data constructor.  This may be more than appear
-in the source code; the extra ones are the existentially quantified
-dictionaries
+These two functions get the real argument types of the constructor,
+without substituting for any type variables.
+
+dataConOrigArgTys returns the arg types of the wrapper, excluding all dictionary args.
+
+dataConRepArgTys retuns the arg types of the worker, including all dictionaries, and
+after any flattening has been done.
 
 \begin{code}
--- Number of type-instantiation arguments
--- All the remaining arguments of the DataCon are (notionally)
--- stored in the DataCon, and are matched in a case expression
-dataConNumInstArgs (MkData {dcTyVars = tyvars}) = length tyvars
+dataConOrigArgTys :: DataCon -> [Type]
+dataConOrigArgTys dc = dcOrigArgTys dc
 
-dataConNumFields (MkData {dcExTheta = theta, dcRepArgTys = arg_tys})
-  = length theta + length arg_tys
+dataConRepArgTys :: DataCon -> [TauType]
+dataConRepArgTys dc = dcRepArgTys dc
+\end{code}
 
-isNullaryDataCon con
-  = dataConNumFields con == 0 -- function of convenience
 
+\begin{code}
 isTupleCon :: DataCon -> Bool
 isTupleCon (MkData {dcTyCon = tc}) = isTupleTyCon tc
        
@@ -409,3 +365,93 @@ isUnboxedTupleCon (MkData {dcTyCon = tc}) = isUnboxedTupleTyCon tc
 isExistentialDataCon :: DataCon -> Bool
 isExistentialDataCon (MkData {dcExTyVars = tvs}) = not (null tvs)
 \end{code}
+
+
+\begin{code}
+classDataCon :: Class -> DataCon
+classDataCon clas = case tyConDataCons (classTyCon clas) of
+                     (dict_constr:no_more) -> ASSERT( null no_more ) dict_constr 
+\end{code}
+
+%************************************************************************
+%*                                                                     *
+\subsection{Splitting products}
+%*                                                                     *
+%************************************************************************
+
+\begin{code}
+splitProductType_maybe
+       :: Type                         -- A product type, perhaps
+       -> Maybe (TyCon,                -- The type constructor
+                 [Type],               -- Type args of the tycon
+                 DataCon,              -- The data constructor
+                 [Type])               -- Its *representation* arg types
+
+       -- Returns (Just ...) for any
+       --      concrete (i.e. constructors visible)
+       --      single-constructor
+       --      not existentially quantified
+       -- type whether a data type or a new type
+       --
+       -- Rejecing existentials is conservative.  Maybe some things
+       -- could be made to work with them, but I'm not going to sweat
+       -- it through till someone finds it's important.
+
+splitProductType_maybe ty
+  = case splitTyConApp_maybe ty of
+       Just (tycon,ty_args)
+          | isProductTyCon tycon       -- Includes check for non-existential,
+                                       -- and for constructors visible
+          -> Just (tycon, ty_args, data_con, dataConArgTys data_con ty_args)
+          where
+             data_con = head (tyConDataConsIfAvailable tycon)
+       other -> Nothing
+
+splitProductType str ty
+  = case splitProductType_maybe ty of
+       Just stuff -> stuff
+       Nothing    -> pprPanic (str ++ ": not a product") (pprType ty)
+
+-- We attempt to unbox/unpack a strict field when either:
+--   (i)  The tycon is imported, and the field is marked '! !', or
+--   (ii) The tycon is defined in this module, the field is marked '!',
+--       and the -funbox-strict-fields flag is on.
+--
+-- This ensures that if we compile some modules with -funbox-strict-fields and
+-- some without, the compiler doesn't get confused about the constructor
+-- representations.
+
+chooseBoxingStrategy :: TyCon -> Type -> StrictnessMark -> StrictnessMark
+       -- Transforms any MarkedUserStricts into MarkUnboxed or MarkedStrict
+chooseBoxingStrategy tycon arg_ty strict
+  = case strict of
+       MarkedUserStrict
+         | opt_UnboxStrictFields
+               && unbox arg_ty -> MarkedUnboxed
+         | otherwise -> MarkedStrict
+       other -> strict
+  where
+       -- beware: repType will go into a loop if we try this on a recursive
+       -- type (for reasons unknown...), hence the check for recursion below.
+    unbox ty =  
+       case splitTyConApp_maybe ty of
+               Nothing -> False
+               Just (arg_tycon, _)
+                 | isRecursiveTyCon arg_tycon -> False
+                 | otherwise ->
+                         case splitTyConApp_maybe (repType ty) of
+                               Nothing -> False
+                               Just (arg_tycon, _) -> isProductTyCon arg_tycon
+
+unbox_strict_arg_ty 
+       :: StrictnessMark       -- After strategy choice; can't be MarkedUserStrict
+       -> Type                 -- Source argument type
+       -> [(Demand,Type)]      -- Representation argument types and demamds
+
+unbox_strict_arg_ty NotMarkedStrict ty = [(lazyDmd, ty)]
+unbox_strict_arg_ty MarkedStrict    ty = [(seqDmd,  ty)]
+unbox_strict_arg_ty MarkedUnboxed   ty 
+  = zipEqual "unbox_strict_arg_ty" (dataConRepStrictness arg_data_con) arg_tys
+  where
+    (_, _, arg_data_con, arg_tys) = splitProductType "unbox_strict_arg_ty" (repType ty)
+\end{code}