Terrific! What are the nofib results? Can we have a couple of artificial benchmarks in cpranal/should_run that show substantial perf improvements because the nested CPR wins in some inner loop? Is https://ghc.haskell.org/trac/ghc/wiki/NestedCPR still an accurate summary of the idea? And the Akio2017 sub-page? It would be easier to review the code if the design documentation accurately described it. I'll look in the new year. Thanks! Simon | -----Original Message----- | From: Matthew Pickering [mailto:matthewtpickering@gmail.com] | Sent: 22 December 2017 17:09 | To: GHC developers <ghc-devs@haskell.org>; Simon Peyton Jones | <simonpj@microsoft.com>; Joachim Breitner <mail@joachim-breitner.de>; | tkn.akio@gmail.com; Sebastian Graf <sgraf1337@gmail.com> | Subject: Nested CPR patch review | | Hi all, | | I recently resurrected akio's nested cpr branch and put it on phabricator | for review. | | https://phabricator.haskell.org/D4244 | | Sebastian has kindly been going over it and ironed out a few kinks in the | last few days. He says now that he believes the patch is correct. | | Is there anything else which needs to be done before merging this patch? | | Simon, would you perhaps be able to give the patch a look over? | | Cheers, | | Matt