-
Notifications
You must be signed in to change notification settings - Fork 5
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Plugin broken on GHC 8.4.2 #43
Comments
I believe this has broken because of changes in GHC's constraint solver in the 8.4.x series. In principle it should be possible to adapt to the constraints that GHC now throws at the plugin, but it will require a bit of digging into the code. |
I ran a bisection on GHC and found the commit there that broke the
|
I encountered quite a few commits that would not build GHC during the bisection that I had to skip. The log I saved as bisect.log. Once I thought to submodule |
@adamgundry, seeing how GHC changed, how might we change the plugin to keep up? |
Thanks for investigating this; I'm sorry for letting this linger on, but it unfortunately requires rather a lot of detailed investigation to understand what the flattened constraints look like and how it might be possible to unflatten and solve them by unification. There is actually WIP in GHC to simplify flattening (https://gitlab.haskell.org/ghc/ghc/-/merge_requests/4149). I haven't tried, but I imagine that patch may change things again, possibly in a good way? |
Closing as we now support 9.0 or later successfully, thanks to the removal of flattening and to |
There are lots of unsolved constraints involving flattening skolems. Perhaps Trac #15147 is related?
The text was updated successfully, but these errors were encountered: