-
Notifications
You must be signed in to change notification settings - Fork 4
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
Awesome! I've got questions! #1
Comments
How's this?
I'm going to do some quick work on profiling to get better timings for how long it takes to call
Yes, it's similar, but the point of this to drive the build in
I dunno 😄 I don't have a good answer for that.
It does, doesn't it?
Yes, this is exactly the idea. The end goal is to have this as something you can drop in to Let me know if that helps, and if you have any other questions! |
Am I reading those timing information right? I'd eventually like to try this out with haskell.nix once we have recursive nix proper in nix. Especially if we can somehow get better performance. |
Looks like it, yep. But I'm not too bothered by the numbers yet. I'm sure it will get faster than this! For one, I'm not even certain if my whole 'fork a GHC for each file is necessary'. Maybe we can act more like GHC --make with just one process |
I'm absolutely looking forward to it! |
Would this require some modifications to GHC itself to have it call nix-build internally? |
That's certainly one option, but unlikely to happen. I think I was more thinking about having |
I've done some work improving performance. Here's the updated times:
I'm using:
to time 1 and
to time 2 & 3. out of this branch: https://github.com/matthewbauer/fused-effects/tree/add-flake I'm not sure why ghc-nix is slightly faster than cabal without cache... Might be that in nix |
This looks really neat. Here's a few questions I have though:
# comment
, to prevent pre-existing cache-hits.Setup.hs
?Thanks! Amazing work!
The text was updated successfully, but these errors were encountered: