Commit e4f98297 authored by Ben Gamari's avatar Ben Gamari 🐢 Committed by Andrey Mokhov

Fix usage of -with-rtsopts (#429)

When I added `-qg` to the default RTS options in
57cfa03c (#385) I neglected to consider that it
the -with-rtsopts flag would override the previous flag setting `-I0`. This
had the effect of reenabling idle GC, causing GC time to regress terribly. I
likely didn't notice this since I had passed the flags directly to the `hadrian`
executable with `+RTS` while testing.

Moreover, I mistakenly wrote `-qg0`, which (somewhat confusingly)
actually *enables* parallel GC. Instead I wanted to write `-qg`.
parent 03ebefdf
......@@ -133,10 +133,9 @@ executable hadrian
-Wredundant-constraints
-fno-warn-name-shadowing
-rtsopts
-- Disable idle GC to avoid redundant GCs while waiting
-- for external processes
-with-rtsopts=-I0
-- Don't use parallel GC as the synchronization time tends to eat any
-- benefit.
-with-rtsopts=-qg0
-- * -I0: Disable idle GC to avoid redundant GCs while
-- waiting for external processes
-- * -qg: Don't use parallel GC as the synchronization
-- time tends to eat any benefit.
"-with-rtsopts=-I0 -qg"
-threaded
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment