[project @ 2003-07-28 16:05:30 by simonmar]
[ghc-hetmet.git] / ghc / compiler / codeGen / CgCon.lhs
index b07e524..c83a035 100644 (file)
@@ -26,35 +26,34 @@ import CgBindery    ( getArgAmodes, bindNewToNode,
                          idInfoToAmode, stableAmodeIdInfo,
                          heapIdInfo, CgIdInfo, bindNewToStack
                        )
-import CgStackery      ( mkTaggedVirtStkOffsets, freeStackSlots, 
-                         updateFrameSize
-                       )
+import CgStackery      ( mkVirtStkOffsets, freeStackSlots, updateFrameSize )
 import CgUsages                ( getRealSp, getVirtSp, setRealAndVirtualSp,
                          getSpRelOffset )
 import CgRetConv       ( assignRegs )
 import Constants       ( mAX_INTLIKE, mIN_INTLIKE, mAX_CHARLIKE, mIN_CHARLIKE,
                          mIN_UPD_SIZE )
-import CgHeapery       ( allocDynClosure, inPlaceAllocDynClosure )
-import CgTailCall      ( performReturn, mkStaticAlgReturnCode, doTailCall,
-                         mkUnboxedTupleReturnCode )
+import CgHeapery       ( allocDynClosure )
+import CgTailCall      ( performReturn, mkStaticAlgReturnCode,
+                         returnUnboxedTuple )
 import CLabel          ( mkClosureLabel )
-import ClosureInfo     ( mkConLFInfo, mkLFArgument, closureLFInfo,
-                         layOutDynConstr, layOutDynClosure,
+import ClosureInfo     ( mkConLFInfo, mkLFArgument, layOutDynConstr, 
                          layOutStaticConstr, closureSize, mkStaticClosure
                        )
 import CostCentre      ( currentOrSubsumedCCS, dontCareCCS, CostCentreStack,
                          currentCCS )
-import DataCon         ( DataCon, dataConName, dataConTag, 
+import DataCon         ( DataCon, dataConTag, 
                          isUnboxedTupleCon, isNullaryDataCon, dataConWorkId, 
-                         dataConWrapId, dataConRepArity
+                         dataConName, dataConRepArity
                        )
-import Id              ( Id, idName, idPrimRep )
+import Id              ( Id, idName, idPrimRep, isDeadBinder )
 import Literal         ( Literal(..) )
 import PrelInfo                ( maybeCharLikeCon, maybeIntLikeCon )
 import PrimRep         ( PrimRep(..), isFollowableRep )
 import Unique          ( Uniquable(..) )
 import Util
 import Outputable
+
+import List            ( partition )
 \end{code}
 
 %************************************************************************
@@ -67,9 +66,8 @@ import Outputable
 cgTopRhsCon :: Id              -- Name of thing bound to this RHS
            -> DataCon          -- Id
            -> [StgArg]         -- Args
-           -> SRT
            -> FCode (Id, CgIdInfo)
-cgTopRhsCon id con args srt
+cgTopRhsCon id con args
   = ASSERT( not (isDllConApp con args) )       -- checks for litlit args too
     ASSERT( args `lengthIs` dataConRepArity con )
 
@@ -78,18 +76,20 @@ cgTopRhsCon id con args srt
 
     let
        name          = idName id
-       lf_info       = closureLFInfo closure_info
+       lf_info       = mkConLFInfo con
        closure_label = mkClosureLabel name
        (closure_info, amodes_w_offsets) 
-               = layOutStaticConstr name con getAmodeRep amodes
+               = layOutStaticConstr con getAmodeRep amodes
+       caffy = any stgArgHasCafRefs args
     in
 
        -- BUILD THE OBJECT
     absC (mkStaticClosure
+           closure_label
            closure_info
            dontCareCCS                 -- because it's static data
            (map fst amodes_w_offsets)  -- Sorted into ptrs first, then nonptrs
-           (nonEmptySRT srt)           -- has CAF refs
+           caffy                       -- has CAF refs
          )                                     `thenC`
                -- NOTE: can't use idCafInfo instead of nonEmptySRT above,
                -- because top-level constructors that were floated by
@@ -138,7 +138,7 @@ at all.
 \begin{code}
 buildDynCon binder cc con []
   = returnFC (stableAmodeIdInfo binder
-                               (CLbl (mkClosureLabel (idName (dataConWrapId con))) PtrRep)
+                               (CLbl (mkClosureLabel (dataConName con)) PtrRep)
                                (mkConLFInfo con))
 \end{code}
 
@@ -186,10 +186,10 @@ buildDynCon binder ccs con args
   = allocDynClosure closure_info use_cc blame_cc amodes_w_offsets `thenFC` \ hp_off ->
     returnFC (heapIdInfo binder hp_off lf_info)
   where
-    (closure_info, amodes_w_offsets)
-      = layOutDynClosure (idName binder) getAmodeRep args lf_info NoC_SRT
     lf_info = mkConLFInfo con
 
+    (closure_info, amodes_w_offsets) = layOutDynConstr con getAmodeRep args
+
     use_cc     -- cost-centre to stick in the object
       = if currentOrSubsumedCCS ccs
        then CReg CurCostCentre
@@ -220,10 +220,8 @@ bindConArgs con args
   = ASSERT(not (isUnboxedTupleCon con))
     mapCs bind_arg args_w_offsets
    where
-     bind_arg (arg, offset) = bindNewToNode arg offset mkLFArgument
-     (_, args_w_offsets)    = layOutDynConstr bogus_name con idPrimRep args
-
-bogus_name = panic "bindConArgs"
+     bind_arg (arg, offset) = bindNewToNode arg offset (mkLFArgument arg)
+     (_, args_w_offsets)    = layOutDynConstr con idPrimRep args
 \end{code}
 
 Unboxed tuples are handled slightly differently - the object is
@@ -231,33 +229,51 @@ returned in registers and on the stack instead of the heap.
 
 \begin{code}
 bindUnboxedTupleComponents
-       :: [Id]                                 -- args
-       -> FCode ([MagicId],                    -- regs assigned
-                 [(VirtualSpOffset,Int)],      -- tag slots
-                 Bool)                         -- any components on stack?
+       :: [Id]                         -- Aargs
+       -> FCode ([MagicId],            -- Regs assigned
+                 Int,                  -- Number of pointer stack slots
+                 Int,                  -- Number of non-pointer stack slots
+                 VirtualSpOffset)      -- Offset of return address slot
+                                       -- (= realSP on entry)
 
 bindUnboxedTupleComponents args
- =  -- Assign as many components as possible to registers
+ =      -- Assign as many components as possible to registers
     let (arg_regs, _leftovers) = assignRegs [] (map idPrimRep args)
        (reg_args, stk_args)   = splitAtList arg_regs args
-    in
 
-    -- Allocate the rest on the stack (ToDo: separate out pointers)
+       -- separate the rest of the args into pointers and non-pointers
+       (ptr_args, nptr_args) = 
+          partition (isFollowableRep . idPrimRep) stk_args
+    in
+  
+    -- Allocate the rest on the stack
+    -- The real SP points to the return address, above which any 
+    -- leftover unboxed-tuple components will be allocated
     getVirtSp `thenFC` \ vsp ->
     getRealSp `thenFC` \ rsp ->
-    let (top_sp, stk_offsets, tags) = 
-               mkTaggedVirtStkOffsets rsp idPrimRep stk_args
+    let 
+       (ptr_sp,  ptr_offsets)  = mkVirtStkOffsets rsp    idPrimRep ptr_args
+       (nptr_sp, nptr_offsets) = mkVirtStkOffsets ptr_sp idPrimRep nptr_args
+        ptrs  = ptr_sp - rsp
+       nptrs = nptr_sp - ptr_sp
     in
 
     -- The stack pointer points to the last stack-allocated component
-    setRealAndVirtualSp top_sp                 `thenC`
+    setRealAndVirtualSp nptr_sp                `thenC`
 
-    -- need to explicitly free any empty slots we just jumped over
-    (if vsp < rsp then freeStackSlots [vsp+1 .. rsp] else nopC) `thenC`
+    -- We have just allocated slots starting at real SP + 1, and set the new
+    -- virtual SP to the topmost allocated slot.  
+    -- If the virtual SP started *below* the real SP, we've just jumped over
+    -- some slots that won't be in the free-list, so put them there
+    -- This commonly happens because we've freed the return-address slot
+    -- (trimming back the virtual SP), but the real SP still points to that slot
+    freeStackSlots [vsp+1,vsp+2 .. rsp]                `thenC`
 
     bindArgsToRegs reg_args arg_regs           `thenC`
-    mapCs bindNewToStack stk_offsets           `thenC`
-    returnFC (arg_regs,tags, not (null stk_offsets))
+    mapCs bindNewToStack ptr_offsets           `thenC`
+    mapCs bindNewToStack nptr_offsets          `thenC`
+
+    returnFC (arg_regs, ptrs, nptrs, rsp)
 \end{code}
 
 %************************************************************************
@@ -278,7 +294,7 @@ cgReturnDataCon con amodes
 
     case sequel of
 
-      CaseAlts _ (Just (alts, Just (maybe_deflt, (_,deflt_lbl))))
+      CaseAlts _ (Just (alts, Just (deflt_bndr, (_,deflt_lbl)))) False
        | not (dataConTag con `is_elem` map fst alts)
        ->
                -- Special case!  We're returning a constructor to the default case
@@ -292,78 +308,17 @@ cgReturnDataCon con amodes
                --      if the default is a non-bind-default (ie does not use y),
                --      then we should simply jump to the default join point;
 
-               case maybe_deflt of
-                   Nothing -> performReturn AbsCNop {- No reg assts -} jump_to_join_point
-                   Just _  -> build_it_then jump_to_join_point
+               if isDeadBinder deflt_bndr
+               then performReturn AbsCNop {- No reg assts -} jump_to_join_point
+               else build_it_then jump_to_join_point
        where
          is_elem = isIn "cgReturnDataCon"
          jump_to_join_point sequel = absC (CJump (CLbl deflt_lbl CodePtrRep))
                -- Ignore the sequel: we've already looked at it above
 
-       -- If the sequel is an update frame, we might be able to
-       -- do update in place...
-      UpdateCode
-       |  not (isNullaryDataCon con)  -- no nullary constructors, please
-       && not (any isFollowableRep (map getAmodeRep amodes))
-                                       -- no ptrs please (generational gc...)
-       && closureSize closure_info <= mIN_UPD_SIZE
-                                       -- don't know the real size of the
-                                       -- thunk, so assume mIN_UPD_SIZE
-
-       ->      -- get a new temporary and make it point to the updatee
-          let 
-               uniq = getUnique con
-               temp = CTemp uniq PtrRep 
-          in
-
-          profCtrC SLIT("TICK_UPD_CON_IN_PLACE") 
-                       [mkIntCLit (length amodes)] `thenC`
-
-          getSpRelOffset args_sp                       `thenFC` \ sp_rel ->
-          absC (CAssign temp 
-                   (CMacroExpr PtrRep UPD_FRAME_UPDATEE [CAddr sp_rel])) 
-               `thenC`
-
-               -- stomp all over it with the new constructor
-          inPlaceAllocDynClosure closure_info temp (CReg CurCostCentre) stuff 
-               `thenC`
-
-               -- don't forget to update Su from the update frame
-          absC (CMacroStmt UPDATE_SU_FROM_UPD_FRAME [CAddr sp_rel])  `thenC`
-
-               -- set Node to point to the closure being returned
-               -- (can't be done earlier: node might conflict with amodes)
-          absC (CAssign (CReg node) temp) `thenC`
-
-               -- pop the update frame off the stack, and do the proper
-               -- return.
-          let new_sp = args_sp - updateFrameSize in
-          setEndOfBlockInfo (EndOfBlockInfo new_sp (OnStack new_sp)) $
-          performReturn (AbsCNop) (mkStaticAlgReturnCode con)
-
-       where
-          (closure_info, stuff) 
-                 = layOutDynConstr (dataConName con) con getAmodeRep amodes
-
       other_sequel     -- The usual case
-
-         | isUnboxedTupleCon con ->
-                       -- Return unboxed tuple in registers
-                 let (ret_regs, leftovers) = 
-                        assignRegs [] (map getAmodeRep amodes)
-                 in
-                 profCtrC SLIT("TICK_RET_UNBOXED_TUP") 
-                               [mkIntCLit (length amodes)] `thenC`
-
-                 doTailCall amodes ret_regs 
-                       mkUnboxedTupleReturnCode
-                       (length leftovers)  {- fast args arity -}
-                       AbsCNop {-no pending assigments-}
-                       Nothing {-not a let-no-escape-}
-                       False   {-node doesn't point-}
-               
-          | otherwise ->
-               build_it_then (mkStaticAlgReturnCode con)
+         | isUnboxedTupleCon con -> returnUnboxedTuple amodes
+          | otherwise ->            build_it_then (mkStaticAlgReturnCode con)
 
   where
     move_to_reg :: CAddrMode -> MagicId -> AbstractC
@@ -384,7 +339,7 @@ cgReturnDataCon con amodes
 
 
                -- RETURN
-         profCtrC SLIT("TICK_RET_NEW") [mkIntCLit (length amodes)] `thenC`
+         profCtrC FSLIT("TICK_RET_NEW") [mkIntCLit (length amodes)] `thenC`
          -- could use doTailCall here.
          performReturn (move_to_reg amode node) return
 \end{code}