> dons        2004/02/19 20:14:41 PST
> 
>   Modified files:        (Branch: backend-hacking-branch)
>     ghc/compiler/cmm     PprC.hs 
>   Log:
>   * pprExpr (CmmRegOff) needs offsets wordShift'd, I'm pretty 
> sure (SimonM?)

No - CmmRegOffs are in bytes.  Some registers have type (StgWord*) which
is why they are casted to StgWord before adding the offset.

It was done this way to get the types right (see comment at the top of
pprExpr).

>   * parens around reg +/- offset expressions.

parens shouldn't be necessary here - pprExpr1 is used when parens might
be needed.

Cheers,
        Simon
_______________________________________________
Cvs-ghc mailing list
[EMAIL PROTECTED]
http://www.haskell.org/mailman/listinfo/cvs-ghc

Reply via email to