[GHC] #15519: Minor code refactoring leads to drastic performance degradation
GHC
ghc-devs at haskell.org
Thu Aug 30 08:48:33 UTC 2018
#15519: Minor code refactoring leads to drastic performance degradation
-------------------------------------+-------------------------------------
Reporter: danilo2 | Owner: (none)
Type: bug | Status: new
Priority: highest | Milestone: 8.8.1
Component: Compiler | Version: 8.4.3
Resolution: | Keywords: SpecConstr
Operating System: Unknown/Multiple | Architecture:
| Unknown/Multiple
Type of failure: None/Unknown | Test Case:
Blocked By: | Blocking:
Related Tickets: | Differential Rev(s):
Wiki Page: |
-------------------------------------+-------------------------------------
Comment (by sgraf):
Replying to [comment:21 simonpj]:
> > Yes, exactly. I'm not happy with doing this (or rather having to
anticipate doing this), either.
>
> I've got lost. What exactly is "this"? Are you happy with what I
propose in #15578?
Sorry, I could have been clearer. TLDR; I'm quite happy, the comment was
unrelated to anything you wrote.
In comment:10, I wrote (with a little more context):
> So, the fix to apply in your situation seems to be to eta-expand test1
and omit the INLINE pragma.
That's what @danilo2's comment:15 alludes to when he writes
> I strongly disagree with the sentence that the "fix would be to remove
INLINE pragma" [...] I suspect @sgraf that you didn't mean "fix" but
instead a "dirty workaround for now", but I preferred to emphasize my
worries regarding this matter.
So, by "this" in comment:20, I meant the workaround of manually eta-
expanding and (more importantly) omitting the INLINE pragma. I'm quite
happy about #15578 :) Not sure about consequences for library authors
being used to the old behavior (if they were even aware of it), though.
--
Ticket URL: <http://ghc.haskell.org/trac/ghc/ticket/15519#comment:22>
GHC <http://www.haskell.org/ghc/>
The Glasgow Haskell Compiler
More information about the ghc-tickets
mailing list