1. 09 Mar, 2001 1 commit
  2. 08 Mar, 2001 9 commits
    • qrczak's avatar
      [project @ 2001-03-08 18:05:31 by qrczak] · 0171137c
      qrczak authored
      s/setIdGlobalDetails/sedGlobalIdDetails/, and import it.
      0171137c
    • qrczak's avatar
      [project @ 2001-03-08 18:03:34 by qrczak] · 69eaf10e
      qrczak authored
      Fix names imported from Id.
      69eaf10e
    • simonmar's avatar
      [project @ 2001-03-08 13:42:17 by simonmar] · fb82a168
      simonmar authored
      add expected output
      fb82a168
    • simonmar's avatar
      [project @ 2001-03-08 13:41:31 by simonmar] · 57f63882
      simonmar authored
      update for blocking putMVar
      57f63882
    • simonpj's avatar
      [project @ 2001-03-08 13:33:23 by simonpj] · f0b635a2
      simonpj authored
      Add NameEnv!
      f0b635a2
    • simonpj's avatar
      [project @ 2001-03-08 12:07:38 by simonpj] · 51a571c0
      simonpj authored
      --------------------
      	A major hygiene pass
      	--------------------
      
      1. The main change here is to
      
      	Move what was the "IdFlavour" out of IdInfo,
      	and into the varDetails field of a Var
      
         It was a mess before, because the flavour was a permanent attribute
         of an Id, whereas the rest of the IdInfo was ephemeral.  It's
         all much tidier now.
      
         Main places to look:
      
      	   Var.lhs	Defn of VarDetails
      	   IdInfo.lhs	Defn of GlobalIdDetails
      
         The main remaining infelicity is that SpecPragmaIds are right down
         in Var.lhs, which seems unduly built-in for such an ephemeral thing.
         But that is no worse than before.
      
      
      2. Tidy up the HscMain story a little.  Move mkModDetails from MkIface
         into CoreTidy (where it belongs more nicely)
      
         This was partly forced by (1) above, because I didn't want to make
         DictFun Ids into a separate kind of Id (which is how it was before).
         Not having them separate means we have to keep a list of them right
         through, rather than pull them out of the bindings at the end.
      
      3. Add NameEnv as a separate module (to join NameSet).
      
      4. Remove unnecessary {-# SOURCE #-} imports from FieldLabel.
      51a571c0
    • simonpj's avatar
      [project @ 2001-03-08 11:59:02 by simonpj] · d78151f6
      simonpj authored
      -------------------------
      	Remove function coercions
      	-------------------------
      
          (coerce (T1->T2) (S1->S2) F) E
        ===>
          coerce T2 S2 (F (coerce S1 T1 E))
      
      This is a generally good transformation, but it
      still doesn't solve the problem I was after.  Consider
      
      
        newtype T = MkT (Int -> Int)
      
        p :: T->T;   p = ...
        q :: T;      q = ...
      
        foo :: T
        {-# INLINE foo #-}
        foo = p $ q
      
        f = \y -> ...((coerce (Int->Int) foo) 3)...
      
      
      Trouble is, foo doesn't see the argument because of the coerce, so it
      thinks it's a lone variable and doesn't inline.
      
      Another problem is that since $ ins't inlined into foo's RHS, foo
      looks like a redex, which we are reluctant to inline inside a lambda,
      even with an INLINE pragma.  Maybe we should be bolder?
      
      Anyway, this commit is an improvement to Simplify, but the story is not
      over!
      d78151f6
    • simonmar's avatar
      [project @ 2001-03-08 11:44:16 by simonmar] · 41d0a61d
      simonmar authored
      remove cyclic dependency
      41d0a61d
    • simonmar's avatar
      [project @ 2001-03-08 09:50:18 by simonmar] · 68de0081
      simonmar authored
      rearrange slightly to make this compile again.
      68de0081
  3. 07 Mar, 2001 5 commits
  4. 06 Mar, 2001 6 commits
  5. 05 Mar, 2001 12 commits
    • simonpj's avatar
      [project @ 2001-03-05 15:37:25 by simonpj] · c8276ef7
      simonpj authored
      Exploit the 1-shot lambda HACK in etaExpandArity
      
      We often find code like
      
      	f :: Int -> IO ()
      	f = \ x -> case ... of
      			p1 -> \s -> ...rhs1...
      			p2 -> \s -> ...rhs2...
      
      where the \s is a state transformer lambda.  Almost invariably
      these \s things are one-shot; that is, we virtually never say
      
      	let
      	   h = f 3
      	in
      	h >> h >> h
      
      In this case we'd be much better off eta-expanding f, to
      
      	f :: Int -> IO ()
      	f = \ x \ s -> case ... of
      			   p1 -> ...rhs1...
      			   p2 -> ...rhs2...
      
      GHC already has a MAJOR HACK in
      
      	Id.isOneShotLambda
      
      which declares that any \s::State# T is a one-shot lambda.  It's
      almost always true, and it makes a big difference.
      
      
      This commit simply makes use of isOneShotLambda to improve the
      results of CoreUtils.etaExpandArity.  Which has the desired effect.
      
      There isn't a flag to control the MAJOR HACK yet.  Maybe there should be.
      
      Anyway, some of Manuel's array code should improve a lot.
      c8276ef7
    • simonpj's avatar
      [project @ 2001-03-05 12:46:16 by simonpj] · 757f1ebd
      simonpj authored
      Print debug uniques consistently in base64
      757f1ebd
    • simonpj's avatar
      [project @ 2001-03-05 12:45:45 by simonpj] · e6dff21d
      simonpj authored
      Improve SpecConstr
      
      This commit fixes SpecConstr so that it can see the effect of
      enclosing case expressions properly.  That's what the "cons" field
      in ScEnv is for.
      
      As a result, consider this function:
      
        data AccessPath = Cont  AccessPath
      		  | Value Int
      
        demandAll n ap@(Cont (Value (I# i1)))
          = case n of
      	0     -> i1
      	other -> i1 +# demandAll (n-1) ap
      
      SpecConstr now successfully compiles it to this:
      
        $s$wdemandAll
          = \ i1 :: PrelGHC.Int# sc :: PrelGHC.Int# ->
            case sc of ds {
      	0 -> i1;
      	__DEFAULT -> PrelGHC.+# i1 (Foo.$s$wdemandAll i1 (PrelGHC.-# ds 1))
            }
      
      with the rule
      
       "SC:$wdemandAll1" __forall i1 :: PrelGHC.Int# ,
      			    sc :: PrelGHC.Int# .
      	Foo.$wdemandAll sc (Foo.$wCont (Foo.$wValue (PrelBase.$wI# i1)))
      	= Foo.$s$wdemandAll i1 sc ;
      e6dff21d
    • simonpj's avatar
      [project @ 2001-03-05 12:19:37 by simonpj] · b5cad075
      simonpj authored
      Better dump of transformation rules
      b5cad075
    • simonpj's avatar
      [project @ 2001-03-05 12:18:43 by simonpj] · ee7e91f5
      simonpj authored
      Remove debug trace
      ee7e91f5
    • simonpj's avatar
      [project @ 2001-03-05 12:18:21 by simonpj] · d14c46d3
      simonpj authored
      Make error message more helpful
      d14c46d3
    • simonpj's avatar
      [project @ 2001-03-05 12:18:05 by simonpj] · e71eb9f0
      simonpj authored
      Remove dead isSysOcc
      e71eb9f0
    • simonpj's avatar
      [project @ 2001-03-05 10:34:44 by simonpj] · baf7143a
      simonpj authored
      Add a test that Hugs-Feb-2001 fails
      baf7143a
    • simonmar's avatar
      [project @ 2001-03-05 10:29:00 by simonmar] · 66331490
      simonmar authored
      remove cruft
      66331490
    • simonmar's avatar
      [project @ 2001-03-05 10:06:27 by simonmar] · 9c59d745
      simonmar authored
      small rearrangement
      9c59d745
    • simonmar's avatar
      [project @ 2001-03-05 10:05:58 by simonmar] · b671b141
      simonmar authored
      fix -fgenerics
      b671b141
    • qrczak's avatar
      [project @ 2001-03-05 00:07:23 by qrczak] · 429854e8
      qrczak authored
      Use custom parser monad instead of Parsec. It remembers the text which
      has been parsed, so it needs not to be reconstructed after parsing.
      
      Operators containing '--' are now handled correctly. '#' triggers
      special processing only if it's not a part of an operator, i.e. if
      a varsym token is exactly a single '#'.
      
      Backslash-newline pairs in C lexical world are now handled correctly
      (removed at an early stage).
      
      Option --keep replaced with --no-compile (stop after writing *.hs_make.c).
      429854e8
  6. 04 Mar, 2001 2 commits
  7. 03 Mar, 2001 1 commit
  8. 02 Mar, 2001 4 commits
    • simonmar's avatar
      [project @ 2001-03-02 17:35:20 by simonmar] · 920d0d7e
      simonmar authored
      Fix :type again, by resurrecting typecheckExpr.  Now the expression
      doesn't get the monomorphism restriction applied to it.
      920d0d7e
    • simonmar's avatar
      [project @ 2001-03-02 16:15:53 by simonmar] · 435b1086
      simonmar authored
      ASSERT in updateWithIndirection() that we haven't already updated this
      object with an indirection, and fix two places in the RTS where this
      could happen.
      
      The problem only occurs when we're in a black-hole-style loop, and
      there are multiple update frames on the stack pointing to the same
      object (this is possible because of lazy black-holing).  Both stack
      squeezing and asynchronous exception raising walk down the stack and
      remove update frames, updating their contents with indirections.  If
      we don't protect against multiple updates, the mutable list in the old
      generation may get into a bogus state.
      435b1086
    • simonmar's avatar
      [project @ 2001-03-02 16:12:18 by simonmar] · b126c85f
      simonmar authored
      Add a new closure flag, IND, to identify indirections.
      b126c85f
    • simonmar's avatar
      [project @ 2001-03-02 14:36:16 by simonmar] · ffaa2614
      simonmar authored
      Add some ASSERT()s so we can catch updates where updatee==target.
      ffaa2614