Skip to content

Conversation

@hvr
Copy link
Member

@hvr hvr commented Sep 25, 2014

hvr added a commit to ghc/ghc that referenced this pull request Sep 25, 2014
This removes a couple of `INLINE` pragmas from the generics helper
classes. With this change the compile times and memory usage should
go back to the previous GHC 7.8.3 situation.

This has been submitted upstream as haskell/binary#62
@hvr
Copy link
Member Author

hvr commented Dec 3, 2014

bump

@kolmodin
Copy link
Member

kolmodin commented Dec 3, 2014

Thanks for the reminder.
I wrote a benchmark earlier to compare with/without the INLINEs, it didn't
make any difference, so it can safely be removed.
I'd like to go ahead and push your fix and release a new version. It might
take a little time since it seems that Russia just has blocked access to
github.
I'll return to this shortly.

@hvr
Copy link
Member Author

hvr commented Dec 4, 2014

@kolmodin oh an btw, please try to get a release-candidate of binary ready by mid-december, to go along with the pre-Xmas GHC 7.10.1 RC1

kolmodin added a commit that referenced this pull request Dec 13, 2014
Remove INLINEs from GBinary/GSum methods
@kolmodin kolmodin merged commit b62c01e into haskell:master Dec 13, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants