halfzebra/create-elm-app

Improve build speed with runtime opts

Closed this issue · 8 comments

andys8 commented

TL;DR

Build time can potentially be improved by passing runtimeOptions / RTS opts.

Issue

The Elm 0.19 is fast. People are still experiencing additional performance improvements by passing RTS options, which is now possible without recompiling the compiler.

Flags

"Antew" wrote this

We saw a 40% improvement in build times with +RTS -A128M -H128M -n8m -RTS, though 40% improvement is a lot less dramatic in 0.19!

https://elmlang.slack.com/archives/C13L7S5GR/p1542423683071500?thread_ts=1542398135.069600&cid=C13L7S5GR

More discussion on elm slack #compile-time.

Changes

elm-webpack-loader supports passing runtimeOptions.
https://github.com/elm-community/elm-webpack-loader/tree/25e7588dfbb789f6d30de91f0bd395aed1e04db7#runtimeoptions-default-undefined

halfzebra commented

Hi @andys8!

Thanks for pointing that out!

Would you be interested in working on a PR for this?

antew commented

One way to use it now is through the GHCRTS environment var, you can use that to set the config as well, e.g. export GHCRTS="-A128M -H128M -n8m"

Edit, forgot the link! https://downloads.haskell.org/~ghc/7.4.1/docs/html/users_guide/runtime-control.html#rts-options-environment

  • 👍2
halfzebra commented

Published under next tag

npm install create-elm-app@next

Unfortunately I haven't noticed any improvement or decrease in performance while testing with https://github.com/halfzebra/elm-spa-example

I'll keep this issue open for the next few days; please let me know if there are any problems.

andys8 commented

Jep, thanks.

If we can't see any difference: Should we remove it? I tend to say yes. Chances are it could potentially do damage, while we don't actually benefit.

halfzebra commented

I tend to agree, but it was worth checking it out 🙂

halfzebra commented

Thanks for participating in this little experiment!

I'm closing the issue due to the fact that the additional runtime configuration does not have a significant impact on the compilation.

Please feel free to reopen this issue if you feel like it's necessary!

andys8 commented

The changes are in master and need to be reverted.

halfzebra commented

Thanks for remembering, that's takien care of already 🙂

  • 👍1