patch applied (ghc): Avoid hanging on to old unfoldings; fixes #4367 (compiler space regression)

[Simon Peyton-Jones]

OK fair enough.  I don't feel too strongly

| -----Original Message-----
| From: cvs-ghc-bounces at haskell.org [mailto:cvs-ghc-bounces at haskell.org] On Behalf Of
| Ian Lynagh
| Sent: 24 October 2010 18:40
| To: cvs-ghc at haskell.org
| Subject: Re: patch applied (ghc): Avoid hanging on to old unfoldings; fixes #4367
| (compiler space regression)
| 
| 
| Hi Simon,
| 
| On Thu, Oct 21, 2010 at 07:57:28AM +0000, Simon Peyton-Jones wrote:
| >
| > Don't you think it'd be better to put that `seq` into 'mkUnfolding'
| > rather than into the calls to mkUnfolding?
| 
| Hmm, well, it's at the call site that we are creating (isBottomingId
| id), and trying to avoid holding on to (id), so having the seq where it
| is is clearer to me. We could do it in mkUnfolding itself, but there
| we'd just be forcing (is_bottoming) without it being obvious why forcing
| it is beneficial.
| 
| I don't feel strongly, though; what do you think?
| 
| 
| Thanks
| Ian
| 
| _______________________________________________
| Cvs-ghc mailing list
| Cvs-ghc at haskell.org
| http://www.haskell.org/mailman/listinfo/cvs-ghc