Re: darcs patch: Replace FastInt with unpacked/newtyped Int

2008-02-13 Thread Ian Lynagh
Hi Twan, Thanks for the patches, and for checking the perofrmance implications! On Mon, Feb 04, 2008 at 04:10:29AM +0100, Twan van Laarhoven wrote: > > When also trying to change the FastInt in Var to an unpacked Unique I ran > into a bug (#2070); The record selector realUnique will not be opt

RE: darcs patch: Replace FastInt with unpacked/newtyped Int

2008-02-04 Thread Simon Peyton-Jones
| This patch changes Unique from | |data Unique = MkUnique FastInt | | to | |newtype Unique = MkUnique Int | | Since this is performance critical code, I have checked the compiler output | (core and stg), to check that it is really the same. The only difference is an | improvement. mkUnique