David Feuer
David Feuer
Hmph. That does sound like a slowdown. Checking for overflow would probably be a slowdown too (more work for branch prediction), but what do I know? Changing the documentation is...
The type of the function gives me no clue what it does. That makes me a bit suspicious. Can you write documentation that makes it easy for people to think...
Uh oh.... I assume that second one extracted to a `Set`?
Of course, deleting those uses will prevent us from checking coverage of the relevant paths. Have you filed a GHC ticket? That first thing is an obvious compiler bug!
> This might have been fixed by [haskell/cabal#9464](https://github.com/haskell/cabal/pull/9464) . What is the version of `cabal` that has been used? This *looks* very different; it's complaining about an incomplete pattern match...
I can probably do it sometime tonight, but I won't feel bad if someone beats me to the punch.
@meooow25, issue 1 seems to be confined to the definition of `hetPtrEq`, which we don't actually use anywhere. So just deleting it should be a safe workaround.
@clyring, I've filed the first issue upstream as [#24608](https://gitlab.haskell.org/ghc/ghc/-/issues/24608).
@BinderDavid , ah, you were talking about the second issue. My bad.
@mikeplus64 , such a rewrite rule would probably be impossible to write. `enumFromThenTo` gets rewritten to use `build`, and I don't think the things it's rewritten to are exported. In...