X-Git-Url: http://git.megacz.com/?p=ghc-hetmet.git;a=blobdiff_plain;f=compiler%2FcodeGen%2FCgHeapery.lhs;h=3ff646ca07d534c14186ced02cfcfff2a3dbd97e;hp=65f94d1fa2e12ee6109f4815c7803826d4e05bcc;hb=d0faaa6fa0cecd23c5670fd199e9206275313666;hpb=a02e7f40afc1aab7fe466f949f505c1d7250713d

diff --git a/compiler/codeGen/CgHeapery.lhs b/compiler/codeGen/CgHeapery.lhs
index 65f94d1..3ff646c 100644
--- a/compiler/codeGen/CgHeapery.lhs
+++ b/compiler/codeGen/CgHeapery.lhs
@@ -34,8 +34,8 @@ import CgCallConv
 import ClosureInfo
 import SMRep
 
-import Cmm
-import CmmUtils
+import OldCmm
+import OldCmmUtils
 import Id
 import DataCon
 import TyCon
@@ -433,6 +433,16 @@ do_checks :: WordOff	-- Stack headroom
 	  -> CmmExpr	-- Rts address to jump to on failure
 	  -> Code
 do_checks 0 0 _ _   = nopC
+
+do_checks _ hp _ _
+  | hp > bLOCKS_PER_MBLOCK * bLOCK_SIZE_W
+  = sorry (unlines [
+            "Trying to allocate more than " ++ show (bLOCKS_PER_MBLOCK * bLOCK_SIZE) ++ " bytes.", 
+            "",
+            "See: http://hackage.haskell.org/trac/ghc/ticket/4505",
+            "Suggestion: read data from a file instead of having large static data",
+            "structures in the code."])
+
 do_checks stk hp reg_save_code rts_lbl
   = do_checks' (CmmLit (mkIntCLit (stk*wORD_SIZE))) 
 	       (CmmLit (mkIntCLit (hp*wORD_SIZE)))
@@ -443,22 +453,32 @@ do_checks' :: CmmExpr -> CmmExpr -> Bool -> Bool -> CmmStmts -> CmmExpr -> Code
 do_checks' stk_expr hp_expr stk_nonzero hp_nonzero reg_save_code rts_lbl
   = do	{ doGranAllocate hp_expr
 
-	-- Emit a block for the heap-check-failure code
-	; blk_id <- forkLabelledCode $ do
-			{ whenC hp_nonzero $
-				stmtC (CmmAssign (CmmGlobal HpAlloc) hp_expr)
+        -- The failure block: this saves the registers and jumps to
+        -- the appropriate RTS stub.
+        ; exit_blk_id <- forkLabelledCode $ do {
 			; emitStmts reg_save_code
 			; stmtC (CmmJump rts_lbl []) }
 
+	-- In the case of a heap-check failure, we must also set
+	-- HpAlloc.  NB. HpAlloc is *only* set if Hp has been
+	-- incremented by the heap check, it must not be set in the
+	-- event that a stack check failed, because the RTS stub will
+	-- retreat Hp by HpAlloc.
+	; hp_blk_id <- if hp_nonzero
+                          then forkLabelledCode $ do
+				  stmtC (CmmAssign (CmmGlobal HpAlloc) hp_expr)
+				  stmtC (CmmBranch exit_blk_id)
+                          else return exit_blk_id
+
 	-- Check for stack overflow *FIRST*; otherwise
 	-- we might bumping Hp and then failing stack oflo
 	; whenC stk_nonzero
-		(stmtC (CmmCondBranch stk_oflo blk_id))
+		(stmtC (CmmCondBranch stk_oflo exit_blk_id))
 
 	; whenC hp_nonzero
 		(stmtsC [CmmAssign hpReg 
 				(cmmOffsetExprB (CmmReg hpReg) hp_expr),
-		        CmmCondBranch hp_oflo blk_id]) 
+		        CmmCondBranch hp_oflo hp_blk_id])
 		-- Bump heap pointer, and test for heap exhaustion
 		-- Note that we don't move the heap pointer unless the 
 		-- stack check succeeds.  Otherwise we might end up