[GHC] #14684: combineIdenticalAlts is only partially implemented
GHC
ghc-devs at haskell.org
Fri Mar 30 07:52:45 UTC 2018
#14684: combineIdenticalAlts is only partially implemented
-------------------------------------+-------------------------------------
Reporter: mpickering | Owner: sjakobi
Type: bug | Status: new
Priority: normal | Milestone: 8.6.1
Component: Compiler | Version: 8.2.2
Resolution: | Keywords: newcomer
Operating System: Unknown/Multiple | Architecture:
| Unknown/Multiple
Type of failure: None/Unknown | Test Case:
Blocked By: | Blocking:
Related Tickets: | Differential Rev(s): Phab:D4542
Wiki Page: |
-------------------------------------+-------------------------------------
Comment (by simonpj):
> Should combineAlts get the same optimization as combineIdenticalAlts?
Might combineAlts even be the better place to apply the optimization?
Actually I think you are right -- CSE probably would be a better place.
And then (because it doesn't happen all the time), you can probably do
this stuff regardless of -O2. (Having a flag might still be good just so
you can switch it on and off at will.)
I added some comments to Phab which will be useful either way.
--
Ticket URL: <http://ghc.haskell.org/trac/ghc/ticket/14684#comment:11>
GHC <http://www.haskell.org/ghc/>
The Glasgow Haskell Compiler
More information about the ghc-tickets
mailing list