Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Compiling the benchmarks with 64bit GHC 7.10.3 is bananas #33

Open
thoughtpolice opened this issue Feb 29, 2016 · 3 comments
Open

Compiling the benchmarks with 64bit GHC 7.10.3 is bananas #33

thoughtpolice opened this issue Feb 29, 2016 · 3 comments
Labels
A-bug A-task E-hard E-helpwanted P-medium

Comments

@thoughtpolice
Copy link
Collaborator

I noticed something like 2GB resident and a minute to compile a single module, which I believe was https://github.com/well-typed/binary-serialise-cbor/blob/master/bench/Macro/PkgCereal.hs (I'll double check this).

It's still pretty bad with GHC 7.8 but the compile times for vs-other-libs seems noticeably quicker. We should quantify these slowdowns and investigate a little before reporting bugs upstream.
@thoughtpolice thoughtpolice mentioned this issue Apr 4, 2016
Closed
@thoughtpolice thoughtpolice mentioned this issue May 2, 2016
Closed
@thoughtpolice
Copy link
Collaborator Author

Note: this was partially helped with the fixes for #37, in particular c8f1643.

@thoughtpolice
Copy link
Collaborator Author

This has also been made less bananas with this PR to cereal, which will need to go upstream: GaloisInc/cereal#54

thoughtpolice pushed a commit that referenced this issue Jun 5, 2016
travis/cabal: constrain to cereal >= 0.5.2.0
dc4be43 
This isn't necessary for the actual library, but will take a lot of
stress off of anyone building the benchmarks, including Travis.

With this, we can back out our old hacks to make Travis use a special
copy of cereal for compilation. See issue #33.

Signed-off-by: Austin Seipp <austin@well-typed.com>
@thoughtpolice
Copy link
Collaborator Author

Note: All builds now require cereal >= 0.5.2.0 as mentioned in dc4be43, so this should be improved a lot. But there's still more that could be done I bet.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
A-bug A-task E-hard E-helpwanted P-medium
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@thoughtpolice