RE: new codegen patches

2008-11-27 Thread Simon Peyton-Jones
| > Agreed. I compared the new codegen with the old codegen (on nofib) before | > I left Microsoft, and I recall a minor uptick in compile times. I haven't | > looked into the causes yet. | > | | Are we expecting a /decrease/ in runtime now though? :-) Probably not yet. All of this work has posit

Re: new codegen patches

2008-11-26 Thread Manuel M T Chakravarty
November 2008 09:24 | To: Simon Peyton-Jones | Cc: John Dias; cvs-ghc@haskell.org | Subject: Re: new codegen patches | | Simon Peyton-Jones wrote: | > Oops. where is mkSECAFBlackHoleInfoTableLabel' defined? | | Looks like a merge problem - I removed the SE blackhole variants recently.

Re: new codegen patches

2008-11-26 Thread Don Stewart
dias: > >How's your defence preparation going? > > The defense is finished and went well. I expect to wrap up my revisions > next week. > > >What's the flag to switch on the new path? > > For reasons that are no long obvious to me, there used to be two flags, > but after I submit my next patch

RE: new codegen patches

2008-11-26 Thread John Dias
How's your defence preparation going? The defense is finished and went well. I expect to wrap up my revisions next week. What's the flag to switch on the new path? For reasons that are no long obvious to me, there used to be two flags, but after I submit my next patch (after validating),

RE: new codegen patches

2008-11-26 Thread John Dias
--- | From: Simon Marlow [mailto:[EMAIL PROTECTED] On Behalf Of Simon Marlow | Sent: 26 November 2008 09:24 | To: Simon Peyton-Jones | Cc: John Dias; cvs-ghc@haskell.org | Subject: Re: new codegen patches | | Simon Peyton-Jones wrote: | > Oops. where is mkSECAFBlackHoleInfoTableLabel' defin

RE: new codegen patches

2008-11-26 Thread Simon Peyton-Jones
ah ok I see what to do. I'll send a patch shortly S | -Original Message- | From: Simon Marlow [mailto:[EMAIL PROTECTED] On Behalf Of Simon Marlow | Sent: 26 November 2008 09:24 | To: Simon Peyton-Jones | Cc: John Dias; cvs-ghc@haskell.org | Subject: Re: new codegen patches | |

Re: new codegen patches

2008-11-26 Thread Simon Marlow
Simon Peyton-Jones wrote: Oops. where is mkSECAFBlackHoleInfoTableLabel' defined? Looks like a merge problem - I removed the SE blackhole variants recently. Cheers, Simon Simon /home/simonmar/fp/bin/x86_64-unknown-linux/ghc ... -c cmm/CmmLive.hs -o dist-stage1/build/CmmLive.o -oh

RE: new codegen patches

2008-11-26 Thread Simon Peyton-Jones
Oops. where is mkSECAFBlackHoleInfoTableLabel' defined? Simon /home/simonmar/fp/bin/x86_64-unknown-linux/ghc ... -c cmm/CmmLive.hs -o dist-stage1/build/CmmLive.o -ohi dist-stage1/build/CmmLive.hi codeGen/StgCmmClosure.hs:767:32: Not in scope: `mkSECAFBlackHoleInfoTableLabel' <> | -Or

RE: new codegen patches

2008-11-26 Thread Simon Peyton-Jones
| As you may have noticed, I have finally pushed a large number of patches | for the new code generator. The new codegen is still off by default, so Brilliant thank you. I'll validate myself now. How's your defence preparation going? What's the flag to switch on the new path? I'll look forward