RE: Order of tests in rem and friends

2010-03-29 Thread Simon Peyton-Jones
...@haskell.org [mailto:cvs-ghc-boun...@haskell.org] On Behalf Of | Roman Leshchinskiy | Sent: 29 March 2010 03:03 | To: cvs-ghc@haskell.org list | Subject: Re: Order of tests in rem and friends | | I just realised that we actually have a ticket for this: | http://hackage.haskell.org/trac/ghc/ticket/3065

Re: Order of tests in rem and friends

2010-03-28 Thread Roman Leshchinskiy
I just realised that we actually have a ticket for this: http://hackage.haskell.org/trac/ghc/ticket/3065. Interestingly, that thread reports increased allocations for some nofib programs with a similar patch but I'm not seeing that any longer with 6.13. In any case, we agreed to get rid of the

Order of tests in rem and friends

2010-03-27 Thread Roman Leshchinskiy
Following the discussion in haskell-cafe, I tried changing the order of tests in the Integral methods from: a `rem` b | b == 0 = divZeroError | a == minBound && b == (-1) = overflowError | otherwise = a `remInt` b to a `rem` b |