Add Outputable.blankLine and use it
[ghc-hetmet.git] / compiler / cmm / PprC.hs
index 1a909f2..9f284c8 100644 (file)
@@ -1,3 +1,10 @@
+{-# 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
+
 -----------------------------------------------------------------------------
 --
 -- Pretty-printing of Cmm as C, suitable for feeding gcc
@@ -9,6 +16,8 @@
 --
 -- Print Cmm as real C, for -fvia-C
 --
+-- See wiki:Commentary/Compiler/Backends/PprC
+--
 -- This is simpler than the old PprAbsC, because Cmm is "macro-expanded"
 -- relative to the old AbstractC, and many oddities/decorations have
 -- disappeared from the data type.
@@ -24,9 +33,10 @@ module PprC (
 #include "HsVersions.h"
 
 -- Cmm stuff
+import BlockId
 import Cmm
+import PprCmm  ()      -- Instances only
 import CLabel
-import MachOp
 import ForeignCall
 import ClosureInfo
 
@@ -39,6 +49,8 @@ import UniqFM
 import FastString
 import Outputable
 import Constants
+import BasicTypes
+import CLabel
 
 -- The rest
 import Data.List
@@ -47,11 +59,6 @@ import Data.Char
 import System.IO
 import Data.Word
 
-#ifdef DEBUG
-import PprCmm          () -- instances only
--- import Debug.Trace
-#endif
-
 import Data.Array.ST
 import Control.Monad.ST
 
@@ -71,7 +78,7 @@ pprCs dflags cmms
  = pprCode CStyle (vcat $ map (\c -> split_marker $$ pprC c) cmms)
  where
    split_marker
-     | dopt Opt_SplitObjs dflags = ptext SLIT("__STG_SPLIT_MARKER")
+     | dopt Opt_SplitObjs dflags = ptext (sLit "__STG_SPLIT_MARKER")
      | otherwise                = empty
 
 writeCs :: DynFlags -> Handle -> [RawCmm] -> IO ()
@@ -85,13 +92,13 @@ writeCs dflags handle cmms
 --
 
 pprC :: RawCmm -> SDoc
-pprC (Cmm tops) = vcat $ intersperse (text "") $ map pprTop tops
+pprC (Cmm tops) = vcat $ intersperse blankLine $ map pprTop tops
 
 --
 -- top level procs
 -- 
 pprTop :: RawCmmTop -> SDoc
-pprTop (CmmProc info clbl _params blocks) =
+pprTop (CmmProc info clbl _params (ListGraph blocks)) =
     (if not (null info)
         then pprDataExterns info $$
              pprWordArray (entryLblToInfoLbl clbl) info
@@ -100,7 +107,7 @@ pprTop (CmmProc info clbl _params blocks) =
         [] -> empty
          -- the first block doesn't get a label:
         (BasicBlock _ stmts : rest) -> vcat [
-          text "",
+          blankLine,
           extern_decls,
            (if (externallyVisibleCLabel clbl)
                     then mkFN_ else mkIF_) (pprCLabel clbl) <+> lbrace,
@@ -121,13 +128,13 @@ pprTop (CmmProc info clbl _params blocks) =
 
 pprTop (CmmData _section _ds@[CmmDataLabel lbl, CmmString str]) = 
   hcat [
-    pprLocalness lbl, ptext SLIT("char "), pprCLabel lbl,
-    ptext SLIT("[] = "), pprStringInCStyle str, semi
+    pprLocalness lbl, ptext (sLit "char "), pprCLabel lbl,
+    ptext (sLit "[] = "), pprStringInCStyle str, semi
   ]
 
 pprTop (CmmData _section _ds@[CmmDataLabel lbl, CmmUninitialised size]) = 
   hcat [
-    pprLocalness lbl, ptext SLIT("char "), pprCLabel lbl,
+    pprLocalness lbl, ptext (sLit "char "), pprCLabel lbl,
     brackets (int size), semi
   ]
 
@@ -135,10 +142,16 @@ pprTop top@(CmmData _section (CmmDataLabel lbl : lits)) =
   pprDataExterns lits $$
   pprWordArray lbl lits  
 
+-- Floating info table for safe a foreign call.
+pprTop top@(CmmData _section d@(_ : _))
+  | CmmDataLabel lbl : lits <- reverse d = 
+  let lits' = reverse lits
+  in pprDataExterns lits' $$
+     pprWordArray lbl lits'
+
 -- these shouldn't appear?
 pprTop (CmmData _ _) = panic "PprC.pprTop: can't handle this data"
 
-
 -- --------------------------------------------------------------------------
 -- BasicBlocks are self-contained entities: they always end in a jump.
 --
@@ -161,16 +174,16 @@ pprBBlock (BasicBlock lbl stmts) =
 
 pprWordArray :: CLabel -> [CmmStatic] -> SDoc
 pprWordArray lbl ds
-  = hcat [ pprLocalness lbl, ptext SLIT("StgWord")
-         , space, pprCLabel lbl, ptext SLIT("[] = {") ] 
+  = hcat [ pprLocalness lbl, ptext (sLit "StgWord")
+         , space, pprCLabel lbl, ptext (sLit "[] = {") ] 
     $$ nest 8 (commafy (pprStatics ds))
-    $$ ptext SLIT("};")
+    $$ ptext (sLit "};")
 
 --
 -- has to be static, if it isn't globally visible
 --
 pprLocalness :: CLabel -> SDoc
-pprLocalness lbl | not $ externallyVisibleCLabel lbl = ptext SLIT("static ")
+pprLocalness lbl | not $ externallyVisibleCLabel lbl = ptext (sLit "static ")
                  | otherwise = empty
 
 -- --------------------------------------------------------------------------
@@ -181,46 +194,70 @@ pprStmt :: CmmStmt -> SDoc
 
 pprStmt stmt = case stmt of
     CmmNop       -> empty
-    CmmComment s -> (hang (ptext SLIT("/*")) 3 (ftext s)) $$ ptext SLIT("*/")
+    CmmComment s -> empty -- (hang (ptext (sLit "/*")) 3 (ftext s)) $$ ptext (sLit "*/")
+                          -- XXX if the string contains "*/", we need to fix it
+                          -- XXX we probably want to emit these comments when
+                          -- some debugging option is on.  They can get quite
+                          -- large.
 
     CmmAssign dest src -> pprAssign dest src
 
     CmmStore  dest src
-       | rep == I64 && wordRep /= I64
-       -> ptext SLIT("ASSIGN_Word64") <> 
-               parens (mkP_ <> pprExpr1 dest <> comma <> pprExpr src) <> semi
-
-       | rep == F64 && wordRep /= I64
-       -> ptext SLIT("ASSIGN_DBL") <> 
-               parens (mkP_ <> pprExpr1 dest <> comma <> pprExpr src) <> semi
+       | typeWidth rep == W64 && wordWidth /= W64
+       -> (if isFloatType rep then ptext (sLit "ASSIGN_DBL")
+                              else ptext (sLit ("ASSIGN_Word64"))) <> 
+          parens (mkP_ <> pprExpr1 dest <> comma <> pprExpr src) <> semi
 
        | otherwise
        -> hsep [ pprExpr (CmmLoad dest rep), equals, pprExpr src <> semi ]
        where
-         rep = cmmExprRep src
+         rep = cmmExprType src
 
-    CmmCall (CmmForeignCall fn cconv) results args safety ->
-       -- Controversial: leave this out for now.
-       -- pprUndef fn $$
-
-       pprCall ppr_fn cconv results args safety
+    CmmCall (CmmCallee fn cconv) results args safety ret ->
+        maybe_proto $$
+       fnCall
        where
-       ppr_fn = case fn of
-                  CmmLit (CmmLabel lbl) -> pprCLabel lbl
-                  _ -> parens (cCast (pprCFunType cconv results args) fn)
-                       -- for a dynamic call, cast the expression to
-                       -- a function of the right type (we hope).
-
-       -- we #undef a function before calling it: the FFI is supposed to be
-       -- an interface specifically to C, not to C+CPP.  For one thing, this
-       -- makes the via-C route more compatible with the NCG.  If macros
-       -- are being used for optimisation, then inline functions are probably
-       -- better anyway.
-       pprUndef (CmmLit (CmmLabel lbl)) = 
-          ptext SLIT("#undef") <+> pprCLabel lbl
-       pprUndef _ = empty
-
-    CmmCall (CmmPrim op) results args safety ->
+        cast_fn = parens (cCast (pprCFunType (char '*') cconv results args) fn)
+
+        real_fun_proto lbl = char ';' <> 
+                        pprCFunType (pprCLabel lbl) cconv results args <> 
+                        noreturn_attr <> semi
+
+        fun_proto lbl = ptext (sLit ";EF_(") <>
+                         pprCLabel lbl <> char ')' <> semi
+
+        noreturn_attr = case ret of
+                          CmmNeverReturns -> text "__attribute__ ((noreturn))"
+                          CmmMayReturn    -> empty
+
+        -- See wiki:Commentary/Compiler/Backends/PprC#Prototypes
+       (maybe_proto, fnCall) = 
+            case fn of
+             CmmLit (CmmLabel lbl) 
+                | StdCallConv <- cconv ->
+                    let myCall = pprCall (pprCLabel lbl) cconv results args safety
+                    in (real_fun_proto lbl, myCall)
+                        -- stdcall functions must be declared with
+                        -- a function type, otherwise the C compiler
+                        -- doesn't add the @n suffix to the label.  We
+                        -- can't add the @n suffix ourselves, because
+                        -- it isn't valid C.
+                | CmmNeverReturns <- ret ->
+                    let myCall = pprCall (pprCLabel lbl) cconv results args safety
+                    in (real_fun_proto lbl, myCall)
+                | not (isMathFun lbl) ->
+                    let myCall = braces (
+                                     pprCFunType (char '*' <> text "ghcFunPtr") cconv results args <> semi
+                                  $$ text "ghcFunPtr" <+> equals <+> cast_fn <> semi
+                                  $$ pprCall (text "ghcFunPtr") cconv results args safety <> semi
+                                 )
+                    in (fun_proto lbl, myCall)
+             _ -> 
+                   (empty {- no proto -},
+                    pprCall cast_fn cconv results args safety <> semi)
+                       -- for a dynamic call, no declaration is necessary.
+
+    CmmCall (CmmPrim op) results args safety _ret ->
        pprCall ppr_fn CCallConv results args safety
        where
        ppr_fn = pprCallishMachOp_for_C op
@@ -230,31 +267,29 @@ pprStmt stmt = case stmt of
     CmmJump lbl _params      -> mkJMP_(pprExpr lbl) <> semi
     CmmSwitch arg ids        -> pprSwitch arg ids
 
-pprCFunType :: CCallConv -> CmmHintFormals -> CmmActuals -> SDoc
-pprCFunType cconv ress args
-  = hcat [
-       res_type ress,
-       parens (text (ccallConvAttribute cconv) <>  char '*'),
-       parens (commafy (map arg_type args))
-   ]
+pprCFunType :: SDoc -> CCallConv -> HintedCmmFormals -> HintedCmmActuals -> SDoc
+pprCFunType ppr_fn cconv ress args
+  = res_type ress <+>
+    parens (text (ccallConvAttribute cconv) <>  ppr_fn) <>
+    parens (commafy (map arg_type args))
   where
-       res_type [] = ptext SLIT("void")
-       res_type [(one,hint)] = machRepHintCType (localRegRep one) hint
+       res_type [] = ptext (sLit "void")
+       res_type [CmmHinted one hint] = machRepHintCType (localRegType one) hint
 
-       arg_type (expr,hint) = machRepHintCType (cmmExprRep expr) hint
+       arg_type (CmmHinted expr hint) = machRepHintCType (cmmExprType expr) hint
 
 -- ---------------------------------------------------------------------
 -- unconditional branches
 pprBranch :: BlockId -> SDoc
-pprBranch ident = ptext SLIT("goto") <+> pprBlockId ident <> semi
+pprBranch ident = ptext (sLit "goto") <+> pprBlockId ident <> semi
 
 
 -- ---------------------------------------------------------------------
 -- conditional branches to local labels
 pprCondBranch :: CmmExpr -> BlockId -> SDoc
 pprCondBranch expr ident 
-        = hsep [ ptext SLIT("if") , parens(pprExpr expr) ,
-                        ptext SLIT("goto") , (pprBlockId ident) <> semi ]
+        = hsep [ ptext (sLit "if") , parens(pprExpr expr) ,
+                        ptext (sLit "goto") , (pprBlockId ident) <> semi ]
 
 
 -- ---------------------------------------------------------------------
@@ -271,7 +306,7 @@ pprSwitch e maybe_ids
   = let pairs  = [ (ix, ident) | (ix,Just ident) <- zip [0..] maybe_ids ]
        pairs2 = [ (map fst as, snd (head as)) | as <- groupBy sndEq pairs ]
     in 
-        (hang (ptext SLIT("switch") <+> parens ( pprExpr e ) <+> lbrace)
+        (hang (ptext (sLit "switch") <+> parens ( pprExpr e ) <+> lbrace)
                 4 (vcat ( map caseify pairs2 )))
         $$ rbrace
 
@@ -282,12 +317,12 @@ pprSwitch e maybe_ids
     caseify (ix:ixs, ident) = vcat (map do_fallthrough ixs) $$ final_branch ix
        where 
        do_fallthrough ix =
-                 hsep [ ptext SLIT("case") , pprHexVal ix wordRep <> colon ,
-                        ptext SLIT("/* fall through */") ]
+                 hsep [ ptext (sLit "case") , pprHexVal ix wordWidth <> colon ,
+                        ptext (sLit "/* fall through */") ]
 
        final_branch ix = 
-               hsep [ ptext SLIT("case") , pprHexVal ix wordRep <> colon ,
-                       ptext SLIT("goto") , (pprBlockId ident) <> semi ]
+               hsep [ ptext (sLit "case") , pprHexVal ix wordWidth <> colon ,
+                       ptext (sLit "goto") , (pprBlockId ident) <> semi ]
 
 -- ---------------------------------------------------------------------
 -- Expressions.
@@ -299,7 +334,7 @@ pprSwitch e maybe_ids
 --
 -- has a type in C which is also given by
 --
---     machRepCType (cmmExprRep e)
+--     machRepCType (cmmExprType e)
 --
 -- (similar invariants apply to the rest of the pretty printer).
 
@@ -307,29 +342,8 @@ pprExpr :: CmmExpr -> SDoc
 pprExpr e = case e of
     CmmLit lit -> pprLit lit
 
-    CmmLoad e I64 | wordRep /= I64
-       -> ptext SLIT("PK_Word64") <> parens (mkP_ <> pprExpr1 e)
-
-    CmmLoad e F64 | wordRep /= I64
-       -> ptext SLIT("PK_DBL") <> parens (mkP_ <> pprExpr1 e)
-
-    CmmLoad (CmmReg r) rep 
-       | isPtrReg r && rep == wordRep
-       -> char '*' <> pprAsPtrReg r
-
-    CmmLoad (CmmRegOff r 0) rep 
-       | isPtrReg r && rep == wordRep
-       -> char '*' <> pprAsPtrReg r
-
-    CmmLoad (CmmRegOff r off) rep
-       | isPtrReg r && rep == wordRep 
-       -- ToDo: check that the offset is a word multiple?
-       -> pprAsPtrReg r <> brackets (ppr (off `shiftR` wordShift))
-
-    CmmLoad expr rep ->
-       -- the general case:
-       cLoad expr rep
 
+    CmmLoad e ty -> pprLoad e ty
     CmmReg reg      -> pprCastReg reg
     CmmRegOff reg 0 -> pprCastReg reg
 
@@ -341,6 +355,32 @@ pprExpr e = case e of
 
     CmmMachOp mop args -> pprMachOpApp mop args
 
+
+pprLoad :: CmmExpr -> CmmType -> SDoc
+pprLoad e ty
+  | width == W64, wordWidth /= W64
+  = (if isFloatType ty then ptext (sLit "PK_DBL")
+                      else ptext (sLit "PK_Word64"))
+    <> parens (mkP_ <> pprExpr1 e)
+
+  | otherwise 
+  = case e of
+       CmmReg r | isPtrReg r && width == wordWidth && not (isFloatType ty)
+                -> char '*' <> pprAsPtrReg r
+
+       CmmRegOff r 0 | isPtrReg r && width == wordWidth && not (isFloatType ty)
+                     -> char '*' <> pprAsPtrReg r
+
+       CmmRegOff r off | isPtrReg r && width == wordWidth
+                       , off `rem` wORD_SIZE == 0 && not (isFloatType ty)
+       -- ToDo: check that the offset is a word multiple?
+        --       (For tagging to work, I had to avoid unaligned loads. --ARY)
+                       -> pprAsPtrReg r <> brackets (ppr (off `shiftR` wordShift))
+
+       _other -> cLoad e ty
+  where
+    width = typeWidth ty
+
 pprExpr1 :: CmmExpr -> SDoc
 pprExpr1 (CmmLit lit)    = pprLit1 lit
 pprExpr1 e@(CmmReg _reg)  = pprExpr e
@@ -353,7 +393,7 @@ pprMachOpApp :: MachOp -> [CmmExpr] -> SDoc
 
 pprMachOpApp op args
   | isMulMayOfloOp op
-  = ptext SLIT("mulIntMayOflo") <> parens (commafy (map pprExpr args))
+  = ptext (sLit "mulIntMayOflo") <> parens (commafy (map pprExpr args))
   where isMulMayOfloOp (MO_U_MulMayOflo _) = True
        isMulMayOfloOp (MO_S_MulMayOflo _) = True
        isMulMayOfloOp _ = False
@@ -383,8 +423,15 @@ pprMachOpApp' mop args
     _     -> panic "PprC.pprMachOp : machop with wrong number of args"
 
   where
-    pprArg e | signedOp mop = cCast (machRepSignedCType (cmmExprRep e)) e
+       -- Cast needed for signed integer ops
+    pprArg e | signedOp    mop = cCast (machRep_S_CType (typeWidth (cmmExprType e))) e
+             | needsFCasts mop = cCast (machRep_F_CType (typeWidth (cmmExprType e))) e
             | otherwise    = pprExpr1 e
+    needsFCasts (MO_F_Eq _)   = False
+    needsFCasts (MO_F_Ne _)   = False
+    needsFCasts (MO_F_Neg _)  = True
+    needsFCasts (MO_F_Quot _) = True
+    needsFCasts mop  = floatComparison mop
 
 -- --------------------------------------------------------------------------
 -- Literals
@@ -392,7 +439,18 @@ pprMachOpApp' mop args
 pprLit :: CmmLit -> SDoc
 pprLit lit = case lit of
     CmmInt i rep      -> pprHexVal i rep
-    CmmFloat f rep     -> parens (machRepCType rep) <> (rational f)
+
+    CmmFloat f w       -> parens (machRep_F_CType w) <> str
+        where d = fromRational f :: Double
+              str | isInfinite d && d < 0 = ptext (sLit "-INFINITY")
+                  | isInfinite d          = ptext (sLit "INFINITY")
+                  | isNaN d               = ptext (sLit "NAN")
+                  | otherwise             = text (show d)
+                -- these constants come from <math.h>
+                -- see #1861
+
+    CmmBlock bid       -> mkW_ <> pprCLabelAddr (infoTblLbl bid)
+    CmmHighStackMark   -> panic "PprC printing high stack mark"
     CmmLabel clbl      -> mkW_ <> pprCLabelAddr clbl
     CmmLabelOff clbl i -> mkW_ <> pprCLabelAddr clbl <> char '+' <> int i
     CmmLabelDiffOff clbl1 clbl2 i
@@ -417,23 +475,29 @@ pprLit1 other = pprLit other
 
 pprStatics :: [CmmStatic] -> [SDoc]
 pprStatics [] = []
-pprStatics (CmmStaticLit (CmmFloat f F32) : rest) 
+pprStatics (CmmStaticLit (CmmFloat f W32) : rest) 
+  -- floats are padded to a word, see #1852
+  | wORD_SIZE == 8, CmmStaticLit (CmmInt 0 W32) : rest' <- rest
+  = pprLit1 (floatToWord f) : pprStatics rest'
+  | wORD_SIZE == 4
   = pprLit1 (floatToWord f) : pprStatics rest
-pprStatics (CmmStaticLit (CmmFloat f F64) : rest)
+  | otherwise
+  = pprPanic "pprStatics: float" (vcat (map (\(CmmStaticLit l) -> ppr (cmmLitType l)) rest))
+pprStatics (CmmStaticLit (CmmFloat f W64) : rest)
   = map pprLit1 (doubleToWords f) ++ pprStatics rest
-pprStatics (CmmStaticLit (CmmInt i I64) : rest)
-  | machRepByteWidth I32 == wORD_SIZE
+pprStatics (CmmStaticLit (CmmInt i W64) : rest)
+  | wordWidth == W32
 #ifdef WORDS_BIGENDIAN
-  = pprStatics (CmmStaticLit (CmmInt q I32) : 
-               CmmStaticLit (CmmInt r I32) : rest)
+  = pprStatics (CmmStaticLit (CmmInt q W32) : 
+               CmmStaticLit (CmmInt r W32) : rest)
 #else
-  = pprStatics (CmmStaticLit (CmmInt r I32) : 
-               CmmStaticLit (CmmInt q I32) : rest)
+  = pprStatics (CmmStaticLit (CmmInt r W32) : 
+               CmmStaticLit (CmmInt q W32) : rest)
 #endif
   where r = i .&. 0xffffffff
        q = i `shiftR` 32
-pprStatics (CmmStaticLit (CmmInt i rep) : rest)
-  | machRepByteWidth rep /= wORD_SIZE
+pprStatics (CmmStaticLit (CmmInt i w) : rest)
+  | w /= wordWidth
   = panic "pprStatics: cannot emit a non-word-sized static literal"
 pprStatics (CmmStaticLit lit : rest)
   = pprLit1 lit : pprStatics rest
@@ -444,7 +508,7 @@ pprStatic :: CmmStatic -> SDoc
 pprStatic s = case s of
 
     CmmStaticLit lit   -> nest 4 (pprLit lit)
-    CmmAlign i         -> nest 4 (ptext SLIT("/* align */") <+> int i)
+    CmmAlign i         -> nest 4 (ptext (sLit "/* align */") <+> int i)
     CmmDataLabel clbl  -> pprCLabel clbl <> colon
     CmmUninitialised i -> nest 4 (mkC_ <> brackets (int i))
 
@@ -469,8 +533,8 @@ pprMachOp_for_C mop = case mop of
         -- Integer operations
         MO_Add          _ -> char '+'
         MO_Sub          _ -> char '-'
-        MO_Eq           _ -> ptext SLIT("==")
-        MO_Ne           _ -> ptext SLIT("!=")
+        MO_Eq           _ -> ptext (sLit "==")
+        MO_Ne           _ -> ptext (sLit "!=")
         MO_Mul          _ -> char '*'
 
         MO_S_Quot       _ -> char '/'
@@ -480,51 +544,68 @@ pprMachOp_for_C mop = case mop of
         MO_U_Quot       _ -> char '/'
         MO_U_Rem        _ -> char '%'
 
-        -- Signed comparisons (floating-point comparisons also use these)
-        -- & Unsigned comparisons
-        MO_S_Ge         _ -> ptext SLIT(">=")
-        MO_S_Le         _ -> ptext SLIT("<=")
+        -- & Floating-point operations
+        MO_F_Add        _ -> char '+'
+        MO_F_Sub        _ -> char '-'
+        MO_F_Neg        _ -> char '-'
+        MO_F_Mul        _ -> char '*'
+        MO_F_Quot       _ -> char '/'
+
+        -- Signed comparisons
+        MO_S_Ge         _ -> ptext (sLit ">=")
+        MO_S_Le         _ -> ptext (sLit "<=")
         MO_S_Gt         _ -> char '>'
         MO_S_Lt         _ -> char '<'
 
-        MO_U_Ge         _ -> ptext SLIT(">=")
-        MO_U_Le         _ -> ptext SLIT("<=")
+        -- & Unsigned comparisons
+        MO_U_Ge         _ -> ptext (sLit ">=")
+        MO_U_Le         _ -> ptext (sLit "<=")
         MO_U_Gt         _ -> char '>'
         MO_U_Lt         _ -> char '<'
 
+        -- & Floating-point comparisons
+        MO_F_Eq         _ -> ptext (sLit "==")
+        MO_F_Ne         _ -> ptext (sLit "!=")
+        MO_F_Ge         _ -> ptext (sLit ">=")
+        MO_F_Le         _ -> ptext (sLit "<=")
+        MO_F_Gt         _ -> char '>'
+        MO_F_Lt         _ -> char '<'
+
         -- Bitwise operations.  Not all of these may be supported at all
         -- sizes, and only integral MachReps are valid.
         MO_And          _ -> char '&'
         MO_Or           _ -> char '|'
         MO_Xor          _ -> char '^'
         MO_Not          _ -> char '~'
-        MO_Shl          _ -> ptext SLIT("<<")
-        MO_U_Shr        _ -> ptext SLIT(">>") -- unsigned shift right
-        MO_S_Shr        _ -> ptext SLIT(">>") -- signed shift right
+        MO_Shl          _ -> ptext (sLit "<<")
+        MO_U_Shr        _ -> ptext (sLit ">>") -- unsigned shift right
+        MO_S_Shr        _ -> ptext (sLit ">>") -- signed shift right
 
--- Conversions.  Some of these will be NOPs.
+-- Conversions.  Some of these will be NOPs, but never those that convert
+-- between ints and floats.
 -- Floating-point conversions use the signed variant.
 -- We won't know to generate (void*) casts here, but maybe from
 -- context elsewhere
 
 -- noop casts
-        MO_U_Conv I8 I8     -> empty
-        MO_U_Conv I16 I16   -> empty
-        MO_U_Conv I32 I32   -> empty
-        MO_U_Conv I64 I64   -> empty
-        MO_U_Conv I128 I128 -> empty
-        MO_S_Conv I8 I8     -> empty
-        MO_S_Conv I16 I16   -> empty
-        MO_S_Conv I32 I32   -> empty
-        MO_S_Conv I64 I64   -> empty
-        MO_S_Conv I128 I128 -> empty
-
-       MO_U_Conv _from to  -> parens (machRepCType to)
-       MO_S_Conv _from to  -> parens (machRepSignedCType to)
-
-        _ -> panic "PprC.pprMachOp_for_C: unknown machop"
-
-signedOp :: MachOp -> Bool
+        MO_UU_Conv from to | from == to -> empty
+       MO_UU_Conv _from to  -> parens (machRep_U_CType to)
+
+        MO_SS_Conv from to | from == to -> empty
+       MO_SS_Conv _from to  -> parens (machRep_S_CType to)
+
+        -- TEMPORARY: the old code didn't check this case, so let's leave it out
+        -- to facilitate comparisons against the old output code.
+        --MO_FF_Conv from to | from == to -> empty
+       MO_FF_Conv _from to  -> parens (machRep_F_CType to)
+
+       MO_SF_Conv _from to  -> parens (machRep_F_CType to)
+       MO_FS_Conv _from to  -> parens (machRep_S_CType to)
+
+        _ -> pprTrace "offending mop" (ptext $ sLit $ show mop) $
+             panic "PprC.pprMachOp_for_C: unknown machop"
+
+signedOp :: MachOp -> Bool     -- Argument type(s) are signed ints
 signedOp (MO_S_Quot _)  = True
 signedOp (MO_S_Rem  _)  = True
 signedOp (MO_S_Neg  _)  = True
@@ -533,9 +614,19 @@ signedOp (MO_S_Le   _)      = True
 signedOp (MO_S_Gt   _)  = True
 signedOp (MO_S_Lt   _)  = True
 signedOp (MO_S_Shr  _)  = True
-signedOp (MO_S_Conv _ _) = True
+signedOp (MO_SS_Conv _ _) = True
+signedOp (MO_SF_Conv _ _) = True
 signedOp _ = False
 
+floatComparison :: MachOp -> Bool  -- comparison between float args
+floatComparison (MO_F_Eq   _)   = True
+floatComparison (MO_F_Ne   _)   = True
+floatComparison (MO_F_Ge   _)   = True
+floatComparison (MO_F_Le   _)   = True
+floatComparison (MO_F_Gt   _)   = True
+floatComparison (MO_F_Lt   _)   = True
+floatComparison _ = False
+
 -- ---------------------------------------------------------------------
 -- tend to be implemented by foreign calls
 
@@ -543,33 +634,33 @@ pprCallishMachOp_for_C :: CallishMachOp -> SDoc
 
 pprCallishMachOp_for_C mop 
     = case mop of
-        MO_F64_Pwr  -> ptext SLIT("pow")
-        MO_F64_Sin  -> ptext SLIT("sin")
-        MO_F64_Cos  -> ptext SLIT("cos")
-        MO_F64_Tan  -> ptext SLIT("tan")
-        MO_F64_Sinh -> ptext SLIT("sinh")
-        MO_F64_Cosh -> ptext SLIT("cosh")
-        MO_F64_Tanh -> ptext SLIT("tanh")
-        MO_F64_Asin -> ptext SLIT("asin")
-        MO_F64_Acos -> ptext SLIT("acos")
-        MO_F64_Atan -> ptext SLIT("atan")
-        MO_F64_Log  -> ptext SLIT("log")
-        MO_F64_Exp  -> ptext SLIT("exp")
-        MO_F64_Sqrt -> ptext SLIT("sqrt")
-        MO_F32_Pwr  -> ptext SLIT("powf")
-        MO_F32_Sin  -> ptext SLIT("sinf")
-        MO_F32_Cos  -> ptext SLIT("cosf")
-        MO_F32_Tan  -> ptext SLIT("tanf")
-        MO_F32_Sinh -> ptext SLIT("sinhf")
-        MO_F32_Cosh -> ptext SLIT("coshf")
-        MO_F32_Tanh -> ptext SLIT("tanhf")
-        MO_F32_Asin -> ptext SLIT("asinf")
-        MO_F32_Acos -> ptext SLIT("acosf")
-        MO_F32_Atan -> ptext SLIT("atanf")
-        MO_F32_Log  -> ptext SLIT("logf")
-        MO_F32_Exp  -> ptext SLIT("expf")
-        MO_F32_Sqrt -> ptext SLIT("sqrtf")
-       MO_WriteBarrier -> ptext SLIT("write_barrier")
+        MO_F64_Pwr  -> ptext (sLit "pow")
+        MO_F64_Sin  -> ptext (sLit "sin")
+        MO_F64_Cos  -> ptext (sLit "cos")
+        MO_F64_Tan  -> ptext (sLit "tan")
+        MO_F64_Sinh -> ptext (sLit "sinh")
+        MO_F64_Cosh -> ptext (sLit "cosh")
+        MO_F64_Tanh -> ptext (sLit "tanh")
+        MO_F64_Asin -> ptext (sLit "asin")
+        MO_F64_Acos -> ptext (sLit "acos")
+        MO_F64_Atan -> ptext (sLit "atan")
+        MO_F64_Log  -> ptext (sLit "log")
+        MO_F64_Exp  -> ptext (sLit "exp")
+        MO_F64_Sqrt -> ptext (sLit "sqrt")
+        MO_F32_Pwr  -> ptext (sLit "powf")
+        MO_F32_Sin  -> ptext (sLit "sinf")
+        MO_F32_Cos  -> ptext (sLit "cosf")
+        MO_F32_Tan  -> ptext (sLit "tanf")
+        MO_F32_Sinh -> ptext (sLit "sinhf")
+        MO_F32_Cosh -> ptext (sLit "coshf")
+        MO_F32_Tanh -> ptext (sLit "tanhf")
+        MO_F32_Asin -> ptext (sLit "asinf")
+        MO_F32_Acos -> ptext (sLit "acosf")
+        MO_F32_Atan -> ptext (sLit "atanf")
+        MO_F32_Log  -> ptext (sLit "logf")
+        MO_F32_Exp  -> ptext (sLit "expf")
+        MO_F32_Sqrt -> ptext (sLit "sqrtf")
+       MO_WriteBarrier -> ptext (sLit "write_barrier")
 
 -- ---------------------------------------------------------------------
 -- Useful #defines
@@ -577,33 +668,22 @@ pprCallishMachOp_for_C mop
 
 mkJMP_, mkFN_, mkIF_ :: SDoc -> SDoc
 
-mkJMP_ i = ptext SLIT("JMP_") <> parens i
-mkFN_  i = ptext SLIT("FN_")  <> parens i -- externally visible function
-mkIF_  i = ptext SLIT("IF_")  <> parens i -- locally visible
+mkJMP_ i = ptext (sLit "JMP_") <> parens i
+mkFN_  i = ptext (sLit "FN_")  <> parens i -- externally visible function
+mkIF_  i = ptext (sLit "IF_")  <> parens i -- locally visible
 
 
 mkFB_, mkFE_ :: SDoc
-mkFB_ = ptext SLIT("FB_") -- function code begin
-mkFE_ = ptext SLIT("FE_") -- function code end
+mkFB_ = ptext (sLit "FB_") -- function code begin
+mkFE_ = ptext (sLit "FE_") -- function code end
 
 -- from includes/Stg.h
 --
-mkC_,mkW_,mkP_,mkPP_,mkI_,mkA_,mkD_,mkF_,mkB_,mkL_,mkLI_,mkLW_ :: SDoc
-
-mkC_  = ptext SLIT("(C_)")        -- StgChar
-mkW_  = ptext SLIT("(W_)")        -- StgWord
-mkP_  = ptext SLIT("(P_)")        -- StgWord*
-mkPP_ = ptext SLIT("(PP_)")       -- P_*
-mkI_  = ptext SLIT("(I_)")        -- StgInt
-mkA_  = ptext SLIT("(A_)")        -- StgAddr
-mkD_  = ptext SLIT("(D_)")        -- const StgWord*
-mkF_  = ptext SLIT("(F_)")        -- StgFunPtr
-mkB_  = ptext SLIT("(B_)")        -- StgByteArray
-mkL_  = ptext SLIT("(L_)")        -- StgClosurePtr
-
-mkLI_ = ptext SLIT("(LI_)")       -- StgInt64
-mkLW_ = ptext SLIT("(LW_)")       -- StgWord64
+mkC_,mkW_,mkP_ :: SDoc
 
+mkC_  = ptext (sLit "(C_)")        -- StgChar
+mkW_  = ptext (sLit "(W_)")        -- StgWord
+mkP_  = ptext (sLit "(P_)")        -- StgWord*
 
 -- ---------------------------------------------------------------------
 --
@@ -632,12 +712,12 @@ pprAssign r1 (CmmRegOff r2 off)
 -- We can't cast the lvalue, so we have to cast the rhs if necessary.  Casting
 -- the lvalue elicits a warning from new GCC versions (3.4+).
 pprAssign r1 r2
-  | isFixedPtrReg r1
-  = pprReg r1 <> ptext SLIT(" = ") <> mkP_ <> pprExpr1 r2 <> semi
-  | Just ty <- strangeRegType r1
-  = pprReg r1 <> ptext SLIT(" = ") <> parens ty <> pprExpr1 r2 <> semi
-  | otherwise
-  = pprReg r1 <> ptext SLIT(" = ") <> pprExpr r2 <> semi
+  | isFixedPtrReg r1             = mkAssign (mkP_ <> pprExpr1 r2)
+  | Just ty <- strangeRegType r1 = mkAssign (parens ty <> pprExpr1 r2)
+  | otherwise                    = mkAssign (pprExpr r2)
+    where mkAssign x = if r1 == CmmGlobal BaseReg
+                       then ptext (sLit "ASSIGN_BaseReg") <> parens x <> semi
+                       else pprReg r1 <> ptext (sLit " = ") <> x <> semi
 
 -- ---------------------------------------------------------------------
 -- Registers
@@ -654,9 +734,13 @@ isFixedPtrReg (CmmLocal _) = False
 isFixedPtrReg (CmmGlobal r) = isFixedPtrGlobalReg r
 
 -- True if (pprAsPtrReg reg) will give an expression with type StgPtr
+-- JD: THIS IS HORRIBLE AND SHOULD BE RENAMED, AT THE VERY LEAST.
+-- THE GARBAGE WITH THE VNonGcPtr HELPS MATCH THE OLD CODE GENERATOR'S OUTPUT;
+-- I'M NOT SURE IF IT SHOULD REALLY STAY THAT WAY.
 isPtrReg :: CmmReg -> Bool
 isPtrReg (CmmLocal _)              = False
-isPtrReg (CmmGlobal (VanillaReg n)) = True -- if we print via pprAsPtrReg
+isPtrReg (CmmGlobal (VanillaReg n VGcPtr)) = True -- if we print via pprAsPtrReg
+isPtrReg (CmmGlobal (VanillaReg n VNonGcPtr)) = False --if we print via pprAsPtrReg
 isPtrReg (CmmGlobal reg)           = isFixedPtrGlobalReg reg
 
 -- True if this global reg has type StgPtr
@@ -668,7 +752,7 @@ isFixedPtrGlobalReg SpLim   = True
 isFixedPtrGlobalReg _          = False
 
 -- True if in C this register doesn't have the type given by 
--- (machRepCType (cmmRegRep reg)), so it has to be cast.
+-- (machRepCType (cmmRegType reg)), so it has to be cast.
 isStrangeTypeReg :: CmmReg -> Bool
 isStrangeTypeReg (CmmLocal _)  = False
 isStrangeTypeReg (CmmGlobal g)         = isStrangeTypeGlobal g
@@ -680,9 +764,9 @@ isStrangeTypeGlobal BaseReg         = True
 isStrangeTypeGlobal r                  = isFixedPtrGlobalReg r
 
 strangeRegType :: CmmReg -> Maybe SDoc
-strangeRegType (CmmGlobal CurrentTSO) = Just (ptext SLIT("struct StgTSO_ *"))
-strangeRegType (CmmGlobal CurrentNursery) = Just (ptext SLIT("struct bdescr_ *"))
-strangeRegType (CmmGlobal BaseReg) = Just (ptext SLIT("struct StgRegTable_ *"))
+strangeRegType (CmmGlobal CurrentTSO) = Just (ptext (sLit "struct StgTSO_ *"))
+strangeRegType (CmmGlobal CurrentNursery) = Just (ptext (sLit "struct bdescr_ *"))
+strangeRegType (CmmGlobal BaseReg) = Just (ptext (sLit "struct StgRegTable_ *"))
 strangeRegType _ = Nothing
 
 -- pprReg just prints the register name.
@@ -693,33 +777,38 @@ pprReg r = case r of
         CmmGlobal global -> pprGlobalReg global
                
 pprAsPtrReg :: CmmReg -> SDoc
-pprAsPtrReg (CmmGlobal (VanillaReg n)) = char 'R' <> int n <> ptext SLIT(".p")
+pprAsPtrReg (CmmGlobal (VanillaReg n gcp)) 
+  = WARN( gcp /= VGcPtr, ppr n ) char 'R' <> int n <> ptext (sLit ".p")
 pprAsPtrReg other_reg = pprReg other_reg
 
 pprGlobalReg :: GlobalReg -> SDoc
 pprGlobalReg gr = case gr of
-    VanillaReg n   -> char 'R' <> int n  <> ptext SLIT(".w")
+    VanillaReg n _ -> char 'R' <> int n  <> ptext (sLit ".w")
+       -- pprGlobalReg prints a VanillaReg as a .w regardless
+       -- Example:     R1.w = R1.w & (-0x8UL);
+       --              JMP_(*R1.p);
     FloatReg   n   -> char 'F' <> int n
     DoubleReg  n   -> char 'D' <> int n
     LongReg    n   -> char 'L' <> int n
-    Sp             -> ptext SLIT("Sp")
-    SpLim          -> ptext SLIT("SpLim")
-    Hp             -> ptext SLIT("Hp")
-    HpLim          -> ptext SLIT("HpLim")
-    CurrentTSO     -> ptext SLIT("CurrentTSO")
-    CurrentNursery -> ptext SLIT("CurrentNursery")
-    HpAlloc        -> ptext SLIT("HpAlloc")
-    BaseReg        -> ptext SLIT("BaseReg")
-    GCEnter1       -> ptext SLIT("stg_gc_enter_1")
-    GCFun          -> ptext SLIT("stg_gc_fun")
+    Sp             -> ptext (sLit "Sp")
+    SpLim          -> ptext (sLit "SpLim")
+    Hp             -> ptext (sLit "Hp")
+    HpLim          -> ptext (sLit "HpLim")
+    CurrentTSO     -> ptext (sLit "CurrentTSO")
+    CurrentNursery -> ptext (sLit "CurrentNursery")
+    HpAlloc        -> ptext (sLit "HpAlloc")
+    BaseReg        -> ptext (sLit "BaseReg")
+    EagerBlackholeInfo -> ptext (sLit "stg_EAGER_BLACKHOLE_info")
+    GCEnter1       -> ptext (sLit "stg_gc_enter_1")
+    GCFun          -> ptext (sLit "stg_gc_fun")
 
 pprLocalReg :: LocalReg -> SDoc
-pprLocalReg (LocalReg uniq _ _) = char '_' <> ppr uniq
+pprLocalReg (LocalReg uniq _) = char '_' <> ppr uniq
 
 -- -----------------------------------------------------------------------------
 -- Foreign Calls
 
-pprCall :: SDoc -> CCallConv -> CmmHintFormals -> CmmActuals -> CmmSafety
+pprCall :: SDoc -> CCallConv -> HintedCmmFormals -> HintedCmmActuals -> CmmSafety
        -> SDoc
 
 pprCall ppr_fn cconv results args _
@@ -736,33 +825,33 @@ pprCall ppr_fn cconv results args _
        -- machine registers that are also used for passing arguments in the
        -- C calling convention.
     (if (not opt_Unregisterised) 
-       then ptext SLIT("__DISCARD__();") 
+       then ptext (sLit "__DISCARD__();") 
        else empty) $$
 #endif
     ppr_assign results (ppr_fn <> parens (commafy (map pprArg args))) <> semi
   where 
      ppr_assign []           rhs = rhs
-     ppr_assign [(one,hint)] rhs
-        = pprLocalReg one <> ptext SLIT(" = ")
-                <> pprUnHint hint (localRegRep one) <> rhs
+     ppr_assign [CmmHinted one hint] rhs
+        = pprLocalReg one <> ptext (sLit " = ")
+                <> pprUnHint hint (localRegType one) <> rhs
      ppr_assign _other _rhs = panic "pprCall: multiple results"
 
-     pprArg (expr, PtrHint)
-       = cCast (ptext SLIT("void *")) expr
+     pprArg (CmmHinted expr AddrHint)
+       = cCast (ptext (sLit "void *")) expr
        -- see comment by machRepHintCType below
-     pprArg (expr, SignedHint)
-       = cCast (machRepSignedCType (cmmExprRep expr)) expr
-     pprArg (expr, _other)
+     pprArg (CmmHinted expr SignedHint)
+       = cCast (machRep_S_CType $ typeWidth $ cmmExprType expr) expr
+     pprArg (CmmHinted expr _other)
        = pprExpr expr
 
-     pprUnHint PtrHint    rep = parens (machRepCType rep)
+     pprUnHint AddrHint   rep = parens (machRepCType rep)
      pprUnHint SignedHint rep = parens (machRepCType rep)
      pprUnHint _          _   = empty
 
 pprGlobalRegName :: GlobalReg -> SDoc
 pprGlobalRegName gr = case gr of
-    VanillaReg n   -> char 'R' <> int n  -- without the .w suffix
-    _              -> pprGlobalReg gr
+    VanillaReg n _  -> char 'R' <> int n  -- without the .w suffix
+    _               -> pprGlobalReg gr
 
 -- Currently we only have these two calling conventions, but this might
 -- change in the future...
@@ -775,7 +864,7 @@ is_cish StdCallConv = True
 -- 
 pprTempAndExternDecls :: [CmmBasicBlock] -> (SDoc{-temps-}, SDoc{-externs-})
 pprTempAndExternDecls stmts 
-  = (vcat (map pprTempDecl (eltsUFM temps)), 
+  = (vcat (map pprTempDecl (uniqSetToList temps)), 
      vcat (map (pprExternDecl False{-ToDo-}) (keysFM lbls)))
   where (temps, lbls) = runTE (mapM_ te_BB stmts)
 
@@ -785,28 +874,32 @@ pprDataExterns statics
   where (_, lbls) = runTE (mapM_ te_Static statics)
 
 pprTempDecl :: LocalReg -> SDoc
-pprTempDecl l@(LocalReg _ rep _)
+pprTempDecl l@(LocalReg _ rep)
   = hcat [ machRepCType rep, space, pprLocalReg l, semi ]
 
 pprExternDecl :: Bool -> CLabel -> SDoc
 pprExternDecl in_srt lbl
   -- do not print anything for "known external" things
   | not (needsCDecl lbl) = empty
-  | otherwise              = 
-       hcat [ visibility, label_type (labelType lbl), 
-              lparen, dyn_wrapper (pprCLabel lbl), text ");" ]
+  | Just sz <- foreignLabelStdcallInfo lbl = stdcall_decl sz
+  | otherwise =
+       hcat [ visibility, label_type lbl,
+              lparen, pprCLabel lbl, text ");" ]
  where
-  dyn_wrapper d
-    | in_srt && labelDynamic lbl = text "DLL_IMPORT_DATA_VAR" <> parens d
-    | otherwise                         = d
-
-  label_type CodeLabel = ptext SLIT("F_")
-  label_type DataLabel = ptext SLIT("I_")
+  label_type lbl | isCFunctionLabel lbl = ptext (sLit "F_")
+                | otherwise            = ptext (sLit "I_")
 
   visibility
      | externallyVisibleCLabel lbl = char 'E'
      | otherwise                  = char 'I'
 
+  -- If the label we want to refer to is a stdcall function (on Windows) then
+  -- we must generate an appropriate prototype for it, so that the C compiler will
+  -- add the @n suffix to the label (#2276)
+  stdcall_decl sz =
+        ptext (sLit "extern __attribute__((stdcall)) void ") <> pprCLabel lbl
+        <> parens (commafy (replicate (sz `quot` wORD_SIZE) (machRep_U_CType wordWidth)))
+        <> semi
 
 type TEState = (UniqSet LocalReg, FiniteMap CLabel ())
 newtype TE a = TE { unTE :: TEState -> (a, TEState) }
@@ -840,8 +933,8 @@ te_Lit _ = return ()
 te_Stmt :: CmmStmt -> TE ()
 te_Stmt (CmmAssign r e)                = te_Reg r >> te_Expr e
 te_Stmt (CmmStore l r)         = te_Expr l >> te_Expr r
-te_Stmt (CmmCall _ rs es _)    = mapM_ (te_temp.fst) rs >>
-                                 mapM_ (te_Expr.fst) es
+te_Stmt (CmmCall _ rs es _ _)  = mapM_ (te_temp.hintlessCmm) rs >>
+                                 mapM_ (te_Expr.hintlessCmm) es
 te_Stmt (CmmCondBranch e _)    = te_Expr e
 te_Stmt (CmmSwitch e _)                = te_Expr e
 te_Stmt (CmmJump e _)          = te_Expr e
@@ -865,53 +958,62 @@ te_Reg _            = return ()
 cCast :: SDoc -> CmmExpr -> SDoc
 cCast ty expr = parens ty <> pprExpr1 expr
 
-cLoad :: CmmExpr -> MachRep -> SDoc
+cLoad :: CmmExpr -> CmmType -> SDoc
 #ifdef BEWARE_LOAD_STORE_ALIGNMENT
 cLoad expr rep =
-    let decl = machRepCType rep <+> ptext SLIT("x") <> semi
-        struct = ptext SLIT("struct") <+> braces (decl)
-        packed_attr = ptext SLIT("__attribute__((packed))")
+    let decl = machRepCType rep <+> ptext (sLit "x") <> semi
+        struct = ptext (sLit "struct") <+> braces (decl)
+        packed_attr = ptext (sLit "__attribute__((packed))")
         cast = parens (struct <+> packed_attr <> char '*')
-    in parens (cast <+> pprExpr1 expr) <> ptext SLIT("->x")
+    in parens (cast <+> pprExpr1 expr) <> ptext (sLit "->x")
 #else
 cLoad expr rep = char '*' <> parens (cCast (machRepPtrCType rep) expr)
 #endif
 
+isCmmWordType :: CmmType -> Bool
+-- True of GcPtrReg/NonGcReg of native word size
+isCmmWordType ty = not (isFloatType ty) 
+                  && typeWidth ty == wordWidth
+
 -- This is for finding the types of foreign call arguments.  For a pointer
 -- argument, we always cast the argument to (void *), to avoid warnings from
 -- the C compiler.
-machRepHintCType :: MachRep -> MachHint -> SDoc
-machRepHintCType rep PtrHint    = ptext SLIT("void *")
-machRepHintCType rep SignedHint = machRepSignedCType rep
+machRepHintCType :: CmmType -> ForeignHint -> SDoc
+machRepHintCType rep AddrHint    = ptext (sLit "void *")
+machRepHintCType rep SignedHint = machRep_S_CType (typeWidth rep)
 machRepHintCType rep _other     = machRepCType rep
 
-machRepPtrCType :: MachRep -> SDoc
-machRepPtrCType r | r == wordRep = ptext SLIT("P_")
-                 | otherwise    = machRepCType r <> char '*'
-
-machRepCType :: MachRep -> SDoc
-machRepCType r | r == wordRep = ptext SLIT("W_")
-              | otherwise    = sized_type
-  where sized_type = case r of
-                       I8      -> ptext SLIT("StgWord8")
-                       I16     -> ptext SLIT("StgWord16")
-                       I32     -> ptext SLIT("StgWord32")
-                       I64     -> ptext SLIT("StgWord64")
-                       F32     -> ptext SLIT("StgFloat") -- ToDo: correct?
-                       F64     -> ptext SLIT("StgDouble")
-                       _  -> panic "machRepCType"
-
-machRepSignedCType :: MachRep -> SDoc
-machRepSignedCType r | r == wordRep = ptext SLIT("I_")
-                     | otherwise    = sized_type
-  where sized_type = case r of
-                       I8      -> ptext SLIT("StgInt8")
-                       I16     -> ptext SLIT("StgInt16")
-                       I32     -> ptext SLIT("StgInt32")
-                       I64     -> ptext SLIT("StgInt64")
-                       F32     -> ptext SLIT("StgFloat") -- ToDo: correct?
-                       F64     -> ptext SLIT("StgDouble")
-                       _ -> panic "machRepCType"
+machRepPtrCType :: CmmType -> SDoc
+machRepPtrCType r | isCmmWordType r = ptext (sLit "P_")
+                 | otherwise       = machRepCType r <> char '*'
+
+machRepCType :: CmmType -> SDoc
+machRepCType ty | isFloatType ty = machRep_F_CType w
+               | otherwise      = machRep_U_CType w
+               where
+                 w = typeWidth ty
+
+machRep_F_CType :: Width -> SDoc
+machRep_F_CType W32 = ptext (sLit "StgFloat") -- ToDo: correct?
+machRep_F_CType W64 = ptext (sLit "StgDouble")
+machRep_F_CType _   = panic "machRep_F_CType"
+
+machRep_U_CType :: Width -> SDoc
+machRep_U_CType w | w == wordWidth = ptext (sLit "W_")
+machRep_U_CType W8  = ptext (sLit "StgWord8")
+machRep_U_CType W16 = ptext (sLit "StgWord16")
+machRep_U_CType W32 = ptext (sLit "StgWord32")
+machRep_U_CType W64 = ptext (sLit "StgWord64")
+machRep_U_CType _   = panic "machRep_U_CType"
+
+machRep_S_CType :: Width -> SDoc
+machRep_S_CType w | w == wordWidth = ptext (sLit "I_")
+machRep_S_CType W8  = ptext (sLit "StgInt8")
+machRep_S_CType W16 = ptext (sLit "StgInt16")
+machRep_S_CType W32 = ptext (sLit "StgInt32")
+machRep_S_CType W64 = ptext (sLit "StgInt64")
+machRep_S_CType _   = panic "machRep_S_CType"
+  
 
 -- ---------------------------------------------------------------------
 -- print strings as valid C strings
@@ -940,8 +1042,8 @@ charToC w =
 -- can safely initialise to static locations.
 
 big_doubles 
-  | machRepByteWidth F64 == 2 * wORD_SIZE  = True
-  | machRepByteWidth F64 == wORD_SIZE      = False
+  | widthInBytes W64 == 2 * wORD_SIZE  = True
+  | widthInBytes W64 == wORD_SIZE      = False
   | otherwise = panic "big_doubles"
 
 castFloatToIntArray :: STUArray s Int Float -> ST s (STUArray s Int Int)
@@ -958,7 +1060,7 @@ floatToWord r
        writeArray arr 0 (fromRational r)
        arr' <- castFloatToIntArray arr
        i <- readArray arr' 0
-       return (CmmInt (toInteger i) wordRep)
+       return (CmmInt (toInteger i) wordWidth)
     )
 
 doubleToWords :: Rational -> [CmmLit]
@@ -970,8 +1072,8 @@ doubleToWords r
        arr' <- castDoubleToIntArray arr
        i1 <- readArray arr' 0
        i2 <- readArray arr' 1
-       return [ CmmInt (toInteger i1) wordRep
-              , CmmInt (toInteger i2) wordRep
+       return [ CmmInt (toInteger i1) wordWidth
+              , CmmInt (toInteger i2) wordWidth
               ]
     )
   | otherwise                          -- doubles are 1 word
@@ -980,24 +1082,24 @@ doubleToWords r
        writeArray arr 0 (fromRational r)
        arr' <- castDoubleToIntArray arr
        i <- readArray arr' 0
-       return [ CmmInt (toInteger i) wordRep ]
+       return [ CmmInt (toInteger i) wordWidth ]
     )
 
 -- ---------------------------------------------------------------------------
 -- Utils
 
 wordShift :: Int
-wordShift = machRepLogWidth wordRep
+wordShift = widthInLog wordWidth
 
 commafy :: [SDoc] -> SDoc
 commafy xs = hsep $ punctuate comma xs
 
 -- Print in C hex format: 0x13fa
-pprHexVal :: Integer -> MachRep -> SDoc
-pprHexVal 0 _ = ptext SLIT("0x0")
+pprHexVal :: Integer -> Width -> SDoc
+pprHexVal 0 _ = ptext (sLit "0x0")
 pprHexVal w rep
-  | w < 0     = parens (char '-' <> ptext SLIT("0x") <> go (-w) <> repsuffix rep)
-  | otherwise = ptext SLIT("0x") <> go w <> repsuffix rep
+  | w < 0     = parens (char '-' <> ptext (sLit "0x") <> go (-w) <> repsuffix rep)
+  | otherwise = ptext (sLit "0x") <> go w <> repsuffix rep
   where
        -- type suffix for literals:
        -- Integer literals are unsigned in Cmm/C.  We explicitly cast to
@@ -1006,9 +1108,9 @@ pprHexVal w rep
        -- warnings about integer overflow from gcc.
 
        -- on 32-bit platforms, add "ULL" to 64-bit literals
-      repsuffix I64 | wORD_SIZE == 4 = ptext SLIT("ULL")
+      repsuffix W64 | wORD_SIZE == 4 = ptext (sLit "ULL")
        -- on 64-bit platforms with 32-bit int, add "L" to 64-bit literals
-      repsuffix I64 | cINT_SIZE == 4 = ptext SLIT("UL")
+      repsuffix W64 | cINT_SIZE == 4 = ptext (sLit "UL")
       repsuffix _ = char 'U'
       
       go 0 = empty