Skip to content
Blame Permalink
  • Daniel Gröber (dxld)'s avatar
    ba8c86ed
    rts: Fix -hT option with profiling rts · ba8c86ed
    Daniel Gröber (dxld) authored and Ben Gamari's avatar Ben Gamari committed 
    In dumpCensus we switch/case on doHeapProfile twice. The second switch
tries to barf on unknown doHeapProfile modes but HEAP_BY_CLOSURE_TYPE is
checked by the first switch and not included in the second.

So when trying to pass -hT to the profiling rts it barfs.

This commit simply merges the two switches into one which fixes this
problem.

(cherry picked from commit ed662901)
(cherry picked from commit bd46fbee)
    ba8c86ed
    rts: Fix -hT option with profiling rts
    Daniel Gröber (dxld) authored and Ben Gamari's avatar Ben Gamari committed 
    In dumpCensus we switch/case on doHeapProfile twice. The second switch
tries to barf on unknown doHeapProfile modes but HEAP_BY_CLOSURE_TYPE is
checked by the first switch and not included in the second.

So when trying to pass -hT to the profiling rts it barfs.

This commit simply merges the two switches into one which fixes this
problem.

(cherry picked from commit ed662901)
(cherry picked from commit bd46fbee)