[commit: ghc] : Add FIXME reminding me to remove PrimOp (95fd4a1)

Max Bolingbroke <[email protected]>
Newsgroups gmane.comp.lang.haskell.cvs.ghc
Message-ID <[email protected]>
Repository : ssh://darcs.haskell.org//srv/darcs/ghc

On branch  : 

http://hackage.haskell.org/trac/ghc/changeset/95fd4a1ae1511892a94cdbdecb2e5e630d75878b

>---------------------------------------------------------------

commit 95fd4a1ae1511892a94cdbdecb2e5e630d75878b
Author: Max Bolingbroke <[email protected]>
Date:   Fri Apr 20 15:15:24 2012 +0100

    Add FIXME reminding me to remove PrimOp

>---------------------------------------------------------------

 compiler/supercompile/Supercompile/Core/Syntax.hs |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/compiler/supercompile/Supercompile/Core/Syntax.hs b/compiler/supercompile/Supercompile/Core/Syntax.hs
index 2e2e6e3..99da7f2 100644
--- a/compiler/supercompile/Supercompile/Core/Syntax.hs
+++ b/compiler/supercompile/Supercompile/Core/Syntax.hs
@@ -128,7 +128,7 @@ data TermF ann = Var Id
                | TyApp (ann (TermF ann)) Type
                | CoApp (ann (TermF ann)) Coercion
                | App (ann (TermF ann)) Id
-               | PrimOp PrimOp [Type] [ann (TermF ann)]     -- FIXME: arguably we have just Vars as arguments for better Tag behaviour (otherwise improving the arguments is hidden by the Tag on the whole PrimOp stack frames)
+               | PrimOp PrimOp [Type] [ann (TermF ann)]     -- FIXME: arguably we have just Vars as arguments for better Tag behaviour (otherwise improving the arguments is hidden by the Tag on the whole PrimOp stack frames). FIXME: in fact, we need to change this because *NOT ALL PRIMOP ARGUMENTS ARE STRICT* (e.g. the lazy polymorphic arguments to newMutVar#, newArray#)
                | Case (ann (TermF ann)) Id Type [AltF ann]  -- NB: unlike GHC, for convenience we allow the list of alternatives to be empty
                | Let Id (ann (TermF ann)) (ann (TermF ann)) -- NB: might bind an unlifted thing, in which case evaluation changes. Unlike GHC, we do NOT assume the RHSes of unlifted bindings are ok-for-speculation.
                | LetRec [(Id, ann (TermF ann))] (ann (TermF ann))
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.