Ticket #2587 (closed bug: fixed)

Opened 5 years ago

Last modified 5 years ago

Optimiser bug with extistentials and GADT's

Reported by: NeilMitchell Owned by:
Priority: normal Milestone:
Component: Compiler Version: 6.9
Keywords: neil.mitchell.2@credit-suisse.com, ganesh.sittampalam@credit-suisse.com Cc:
Operating System: Unknown/Multiple Architecture: Unknown/Multiple
Type of failure: Difficulty: Unknown
Test Case: gadt/T2587 Blocked By:
Blocking: Related Tickets:

Description

Given the program:

{-# LANGUAGE GADTs, ExistentialQuantification #-}
module GadtBug(bug) where

data Existential = forall a . Existential (Gadt a)

data Gadt a where Value :: Gadt Double


bug = [ match undefined | ps <- undefined, _ <- ps ]
  where
        match (Existential _) = undefined
        match (Existential _) = undefined

Using GHC Head from yesterday, compiling without optimisations I get:

C:\Neil>ghc --make GadtBug.hs
[1 of 1] Compiling GadtBug          ( GadtBug.hs, GadtBug.o )

GadtBug.hs:11:8:
    Warning: Pattern match(es) are overlapped
             In the definition of `match': match (Existential _) = ...

With optimisations (-O or -O2) I get:

C:\Neil\paradise-bug-csdg\alone>ghc --make GadtBug.hs -O
[1 of 1] Compiling GadtBug          ( GadtBug.hs, GadtBug.o )

GadtBug.hs:11:8:
    Warning: Pattern match(es) are overlapped
             In the definition of `match': match (Existential _) = ...
ghc: panic! (the 'impossible' happened)
  (GHC version 6.9.20080905 for i386-unknown-mingw32):
        idInfo a{tv ag4} [sk]

Please report this as a GHC bug:  http://www.haskell.org/ghc/reportabug

-- Neil

Change History

Changed 5 years ago by simonpj

  • status changed from new to closed
  • difficulty set to Unknown
  • resolution set to fixed
  • testcase set to gadt/T2587

Excellent example, thank you. Fixed by

Sun Sep 14 04:34:34 PDT 2008  simonpj@microsoft.com
  * Fix Trac #2587: take account of type lets

Simon

Changed 5 years ago by simonmar

  • architecture changed from Unknown to Unknown/Multiple

Changed 5 years ago by simonmar

  • os changed from Unknown to Unknown/Multiple
Note: See TracTickets for help on using tickets.