patch applied (ghc): Add assertion for arity match (checks Trac
#2844)
Simon Marlow
marlowsd at gmail.com
Tue Dec 9 05:34:51 EST 2008
Simon Peyton Jones wrote:
> Mon Dec 8 09:32:41 PST 2008 simonpj at microsoft.com
> * Add assertion for arity match (checks Trac #2844)
>
> The exported arity of a function must match the arity for the
> STG function. Trac #2844 was a pretty obscure manifestation of
> the failure of this invariant. This patch doesn't cure the bug;
> rather it adds an assertion to CoreToStg to check the invariant
> so we should get an earlier and less obscure warning if this
> fails in future.
Hooray for the testsuite and validate: out of 2000+ tests, there was
exactly 1 (ONE) that demonstrated this bug, and it was in a test for the
System.Random library of all things. This serves as a reminder to all of
us: any test failure should be treated as suspicious, even if it doesn't
look at all related to something you changed, and this is exactly why we
have to keep validate clean. I am now a fully paid-up validate convert!
Cheers,
Simon
More information about the Cvs-ghc
mailing list