Ticket #565 (assigned bug: None)

Opened 7 years ago

Last modified 2 weeks ago

overlapping instances & fundeps broken

Reported by: ashley-y Assigned to: nobody (accepted)
Priority: low Milestone: _|_
Component: Compiler (Type checker) Version: 5.00
Severity: normal Keywords:
Cc: Difficulty: Unknown
Test Case: Architecture: Unknown/Multiple
Operating System: Unknown/Multiple

Description (Last modified by igloo)

Consider this:

--
class X a
instance X Bool
instance (Num a) => X a
--

For as long as "instance Num Bool" is not declared, the two instances do 
not de facto overlap. But that's not immediately obvious to GHC, so it will 
complain, at least by default. But I can stop it complaining by passing 
-fallow-overlapping-instances, which I interpret as asking GHC to trust me 
that instances don't actually overlap.

But consider this, with an added dependent argument:

--
class X a b | a -> b
instance X Bool Bool
instance (Num a) => X a Char
--

Now GHC will complain even with -fallow-overlapping-instances. I believe 
this is inappropriate.

So why have the fundep? Well, GHC can still make use of it, and it can still 
calculate the dependent type:

--
class X a b | a -> b where
  {
  foo :: a -> b;
  };

instance X Bool Bool where
  {
  foo a = a;
  };

instance (Num a) => X a Char where
  {
  foo a = 'N';
  }

test = foo True;
--

Without the fundep, GHC cannot calculate 'foo True', since 'instance X Bool 
Bool' is not general enough. This is correct. But with the fundep, GHC will 
complain that it can't prove that the two instances don't conflict for the 
fundep, even with -fallow-overlapping-instances.

I submit that GHC with -fallow-overlapping-instances should not complain 
in this case.

Change History

01/25/07 05:31:38 changed by igloo

  • description changed.
  • testcase changed.
  • difficulty set to Unknown.
  • architecture set to Unknown.
  • milestone set to _|_.
  • os set to Unknown.

09/30/08 08:37:41 changed by simonmar

  • architecture changed from Unknown to Unknown/Multiple.

09/30/08 08:52:02 changed by simonmar

  • os changed from Unknown to Unknown/Multiple.