Thanks, no I was not CC'd. Now I've added myself to ghc-tickets (alongside libraries, ghc-devs, haskell-cafe ... whew ;-) .) An initial inspection of the patch looks fine. I'll go ahead and merge it. However, I remind everyone that the priority should be replacing random entirely. It's slow and known to be a bad algorithm (for the splitting bit). Koen Classen & co had an implementation in the Haskell symposium paper that may be a candidate. On Tue, Feb 4, 2014 at 3:04 AM, Simon Peyton Jones wrote:

Hi Ryan

Are you on this thread? You probably should be!

Simon

| -----Original Message----- | From: ghc-tickets [mailto:ghc-tickets-bounces@haskell.org] On Behalf Of | GHC | Sent: 27 January 2014 18:49 | Cc: ghc-tickets@haskell.org | Subject: Re: [GHC] #8704: Use GHC.Exts.build in randoms, randomRs to | achieve fusion | | #8704: Use GHC.Exts.build in randoms, randomRs to achieve fusion | -------------------------------------+--------------------------------- | --- | Reporter: ion1 | Owner: | Type: feature request | Status: patch | Priority: normal | Milestone: | Component: libraries/random | Version: 7.6.3 | Resolution: | Keywords: fusion | Operating System: Unknown/Multiple | Architecture: | Unknown/Multiple | Type of failure: None/Unknown | Difficulty: Unknown | Test Case: | Blocked By: | Blocking: | Related Tickets: #4218 | -------------------------------------+--------------------------------- | --- | | Comment (by nomeata): | | Thanks. From my POV it is worth adding even if you can't measure a | performance gain; it is still good to know that nice code is being | generated. But of course it is up to Ryan Newton (random maintainer) | to | decide this. | | -- | Ticket URL: http://ghc.haskell.org/trac/ghc/ticket/8704#comment:4 | GHC http://www.haskell.org/ghc/ | The Glasgow Haskell Compiler | _______________________________________________ | ghc-tickets mailing list | ghc-tickets@haskell.org | http://www.haskell.org/mailman/listinfo/ghc-tickets