-
Notifications
You must be signed in to change notification settings - Fork 84
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
GHC 8.6.5 takes minutes to compile Agda's Parser.hs #317
Comments
I think I was too quick to blame 2.0 here. It is also slow with 1.20.1.1. Seems to be a GHC problem that was likely fixed in 8.8: Similar issue: |
Yes, that's the issue I had in mind as well. Good thing it's not entirely on 2.0 Out of curiousity: You say
That seems surprising to me, because it's not like we changed the generation of the automaton. Do you mean .hs file size of .o file size? Could you perhaps look at a diff and give a qualitative assessment which parts have become larger? |
Reason: GHC 8.6 is still slow building the happy-generated Parser.hs, GHC 8.8 does not have this problem. See haskell/happy#317
So for |
Thanks. I now recall why the parser is larger. It's the result of #266 (silly me) and I don't think we should go back. Ironically, the biggest array in there is still |
It is known that the
.hs
files generated by happy 2.0 can be 25% larger than the ones generated by happy 1.20. This is e.g. the case for Agda'sParser.hs
generated from https://github.com/agda/agda/blob/7ea5bfa3d6bd374a2f4a0c40534a515b399787a9/src/full/Agda/Syntax/Parser/Parser.y .However, we are also seeing a compile-time deterioration for some GHCs, e.g. 8.6.5.Anecdotally, GHC 8.6.5 pauses for several minutes on my machine when type-checking the happy-2.0 generated
Parser.hs
. (Type-checking this is usually instantaneous.)The text was updated successfully, but these errors were encountered: