patch applied (ghc): Make some more modules use LazyUniqFM instead of UniqFM

Manuel M T Chakravarty chak at cse.unsw.edu.au
Wed Feb 20 23:59:26 EST 2008


Ian,

This is a bad patch (maybe in conjunction with "Rewrite fixTvSubstEnv  
so it iteratively applies its substition").  The patch goes through  
validate, but it breaks builds using the "devel1" scheme defined in  
build.mk.sample.  This might be due to the different optimisation  
settings, but I'd rather think it is due to -DDEBUG in devel1.  The  
way it breaks is that if you compile with devel1, the stage 1 compiler  
builds fine, but when the stage 1 compiler attempts to compiler base,  
the simplifier loops on the first module (i.e., GHC/Base.lhs).

BTW, I was anyway wondering whether validate shouldn't use -DDEBUG for  
the stage 1 compiler.  It's going to make the build somewhat slower  
(how much?), but would be a good sanity check.

Manuel

>
> Wed Feb  6 17:57:14 PST 2008  Ian Lynagh <igloo at earth.li>
>  UNDO: Make some more modules use LazyUniqFM instead of UniqFM
>  If these modules use UniqFM then we get a stack overflow when  
> compiling
>  modules that use fundeps. I haven't tracked down the actual cause.



More information about the Cvs-ghc mailing list