Skip to content

Data.Array.Base: remove useless SPECIALISE pragmas #11

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

Merged
merged 2 commits into from
Dec 20, 2024

Conversation

sheaf
Copy link
Contributor

@sheaf sheaf commented Dec 16, 2024

These SPECIALISE pragmas don't actually do anything, and starting with GHC 9.14 (with GHC MR !12319) will cause a warning to be emitted. This commit simply removes them.

@July541
Copy link
Contributor

July541 commented Dec 17, 2024

Github moved their ubuntu-latest to ubuntu-24.04 from ubuntu-22.04 recently, which caused haskell-setup failed to install some dependencies, see details here.

These SPECIALISE pragmas don't actually do anything,
and starting with GHC 9.14 cause a warning to be emitted.
This commit simply removes them.
@sheaf
Copy link
Contributor Author

sheaf commented Dec 18, 2024

I have rebased over #12. Please let me know if I need to do anything else here. Thanks!

@sheaf
Copy link
Contributor Author

sheaf commented Dec 20, 2024

gentle ping @July541

@July541 July541 merged commit ac6b17b into haskell:master Dec 20, 2024
24 checks passed
@hellwolf
Copy link

For my own learnings, I got this explanation:

12-20 17:19 hellwolf: The tyvar it specialises is not sufficient to │
resolve any instances, so no type class dictionaries can │
be inlined; there are no gains to be had. │

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants