+%************************************************************************
+%* *
+\subsection{Close type variables}
+%* *
+%************************************************************************
+
+(oclose preds tvs) closes the set of type variables tvs,
+wrt functional dependencies in preds. The result is a superset
+of the argument set. For example, if we have
+ class C a b | a->b where ...
+then
+ oclose [C (x,y) z, C (x,p) q] {x,y} = {x,y,z}
+because if we know x and y then that fixes z.
+
+Using oclose
+~~~~~~~~~~~~
+oclose is used
+
+a) When determining ambiguity. The type
+ forall a,b. C a b => a
+is not ambiguous (given the above class decl for C) because
+a determines b.
+
+b) When generalising a type T. Usually we take FV(T) \ FV(Env),
+but in fact we need
+ FV(T) \ (FV(Env)+)
+where the '+' is the oclosure operation. Notice that we do not
+take FV(T)+. This puzzled me for a bit. Consider
+
+ f = E
+
+and suppose e have that E :: C a b => a, and suppose that b is
+free in the environment. Then we quantify over 'a' only, giving
+the type forall a. C a b => a. Since a->b but we don't have b->a,
+we might have instance decls like
+ instance C Bool Int where ...
+ instance C Char Int where ...
+so knowing that b=Int doesn't fix 'a'; so we quantify over it.
+
+ ---------------
+ A WORRY: ToDo!
+ ---------------
+If we have class C a b => D a b where ....
+ class D a b | a -> b where ...
+and the preds are [C (x,y) z], then we want to see the fd in D,
+even though it is not explicit in C, giving [({x,y},{z})]
+
+Similarly for instance decls? E.g. Suppose we have
+ instance C a b => Eq (T a b) where ...
+and we infer a type t with constraints Eq (T a b) for a particular
+expression, and suppose that 'a' is free in the environment.
+We could generalise to
+ forall b. Eq (T a b) => t
+but if we reduced the constraint, to C a b, we'd see that 'a' determines
+b, so that a better type might be
+ t (with free constraint C a b)
+Perhaps it doesn't matter, because we'll still force b to be a
+particular type at the call sites. Generalising over too many
+variables (provided we don't shadow anything by quantifying over a
+variable that is actually free in the envt) may postpone errors; it
+won't hide them altogether.
+
+
+\begin{code}
+oclose :: [PredType] -> TyVarSet -> TyVarSet
+oclose preds fixed_tvs
+ | null tv_fds = fixed_tvs -- Fast escape hatch for common case
+ | otherwise = loop fixed_tvs
+ where
+ loop fixed_tvs
+ | new_fixed_tvs `subVarSet` fixed_tvs = fixed_tvs
+ | otherwise = loop new_fixed_tvs
+ where
+ new_fixed_tvs = foldl extend fixed_tvs tv_fds
+
+ extend fixed_tvs (ls,rs) | ls `subVarSet` fixed_tvs = fixed_tvs `unionVarSet` rs
+ | otherwise = fixed_tvs
+
+ tv_fds :: [(TyVarSet,TyVarSet)]
+ -- In our example, tv_fds will be [ ({x,y}, {z}), ({x,p},{q}) ]
+ -- Meaning "knowing x,y fixes z, knowing x,p fixes q"
+ tv_fds = [ (tyVarsOfTypes xs, tyVarsOfTypes ys)
+ | ClassP cls tys <- preds, -- Ignore implicit params
+ let (cls_tvs, cls_fds) = classTvsFds cls,
+ fd <- cls_fds,
+ let (xs,ys) = instFD fd cls_tvs tys
+ ]
+\end{code}
+
+\begin{code}
+grow :: [PredType] -> TyVarSet -> TyVarSet
+grow preds fixed_tvs
+ | null preds = fixed_tvs
+ | otherwise = loop fixed_tvs
+ where
+ loop fixed_tvs
+ | new_fixed_tvs `subVarSet` fixed_tvs = fixed_tvs
+ | otherwise = loop new_fixed_tvs
+ where
+ new_fixed_tvs = foldl extend fixed_tvs pred_sets
+
+ extend fixed_tvs pred_tvs
+ | fixed_tvs `intersectsVarSet` pred_tvs = fixed_tvs `unionVarSet` pred_tvs
+ | otherwise = fixed_tvs
+
+ pred_sets = [tyVarsOfPred pred | pred <- preds]
+\end{code}
+
+%************************************************************************
+%* *
+\subsection{Generate equations from functional dependencies}
+%* *
+%************************************************************************
+
+