diff options
| author | John Ericson <John.Ericson@Obsidian.Systems> | 2022-06-01 00:04:26 +0000 |
|---|---|---|
| committer | John Ericson <John.Ericson@Obsidian.Systems> | 2022-06-02 15:42:59 +0000 |
| commit | 6520da955ec003b3f7ba931e81df4c4f9873f185 (patch) | |
| tree | 4530a1a0be48fffa1eb648be988aca57c5bb91d2 /rts | |
| parent | d550d9079d6518938a2e41622b1c3ebf1fb24f59 (diff) | |
| download | haskell-6520da955ec003b3f7ba931e81df4c4f9873f185.tar.gz | |
Split out `GHC.HsToCore.{Breakpoints,Coverage}` and use `SizedSeq`
As proposed in
https://gitlab.haskell.org/ghc/ghc/-/merge_requests/7508#note_432877 and
https://gitlab.haskell.org/ghc/ghc/-/merge_requests/7508#note_434676,
`GHC.HsToCore.Ticks` is about ticks, breakpoints are separate and
backend-specific (only for the bytecode interpreter), and mix entry
writing is just for HPC.
With this split we separate out those interpreter- and HPC-specific
its, and keep the main `GHC.HsToCore.Ticks` agnostic.
Also, instead of passing the reversed list and count around, we use
`SizedSeq` which abstracts over the algorithm. This is much nicer to
avoid noise and prevents bugs.
(The bugs are not just hypothetical! I missed up the reverses on an
earlier draft of this commit.)
Diffstat (limited to 'rts')
| -rw-r--r-- | rts/Hpc.c | 2 |
1 files changed, 1 insertions, 1 deletions
@@ -241,7 +241,7 @@ startupHpc(void) /* * Called on a per-module basis, by a constructor function compiled - * with each module (see GHC.HsToCore.Ticks.hpcInitCode), declaring + * with each module (see GHC.HsToCore.Coverage.hpcInitCode), declaring * where the tix boxes are stored in memory. This memory can be uninitized, * because we will initialize it with either the contents of the tix * file, or all zeros. |