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
| 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