[GHC] #8793: Improve GHC.Event.IntTable performance
GHC
ghc-devs at haskell.org
Sat Jan 2 13:10:53 UTC 2016
#8793: Improve GHC.Event.IntTable performance
-------------------------------------+-------------------------------------
Reporter: cdk | Owner:
Type: task | Status: infoneeded
Priority: normal | Milestone: 8.0.1
Component: Core Libraries | Version: 7.6.3
Resolution: | Keywords:
Operating System: Unknown/Multiple | Architecture:
Type of failure: Runtime | Unknown/Multiple
performance bug | Test Case:
Blocked By: | Blocking:
Related Tickets: | Differential Rev(s):
Wiki Page: |
-------------------------------------+-------------------------------------
Comment (by jscholl):
I tried to collect the relevant changes from the patch, but in most cases
could not reproduce the speedups cdk reported. Only for the {{{lookup}}}
function the runtime changed measurable (it improved even more, around 78%
faster than the current implementation). Pulling the
{{{ForeignPtr}}} out of the {{{IORef}}} could make sense, but I could not
measure an impact on performance.
A thing I find surprising is the first argument in {{{updateWith.go}}},
which seems to be totally unused. Is this the correct behavior? Anyway, I
created a second patch which removes the unused argument (and changes the
first element of the return value to a {{{Bool}}}, as it is just tested
for being {{{Nothing}}}), if it is just a leftover from an earlier version
of the function. The change has no impact on performance, though.
--
Ticket URL: <http://ghc.haskell.org/trac/ghc/ticket/8793#comment:9>
GHC <http://www.haskell.org/ghc/>
The Glasgow Haskell Compiler
More information about the ghc-tickets
mailing list