-
Notifications
You must be signed in to change notification settings - Fork 63
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
Support building with GHC 9.4 #1860
Merged
Merged
Commits on May 26, 2023
-
Configuration menu - View commit details
-
Copy full SHA for 58c2a8a - Browse repository at this point
Copy the full SHA 58c2a8aView commit details -
This contains a variety of tweaks needed to build SAW with GHC 9.4: * GHC 9.4 is more conservative about inferring superclass constraints that arise from functional dependencies (see [this section](https://gitlab.haskell.org/ghc/ghc/-/wikis/migration/9.4?version_id=b60e52482a666d25638d59cd7e86851ddf971dc1#constraints-derived-from-superclasses) of the GHC 9.4 Migration Guide), so we must add explicit `m ~ Identity` constraints to certain parts of `heapster-saw` to make it compile with GHC 9.4. * I raised the upper version bounds on `aeson` and `vector` to allow building them with GHC 9.4. * The following submodule changes were brought in to support building with GHC 9.4: * `argo`: #193 * `crucible`: GaloisInc/crucible#1073 (This also requires bumping the `llvm-pretty`, `llvm-pretty-bc-parser`, and `what4` submodules as a side effect) * `language-sally`: GaloisInc/language-sally#13 * `macaw`: GaloisInc/macaw#330 * `parameterized-utils`: GaloisInc/parameterized-utils#146 Fixes #1852.
Configuration menu - View commit details
-
Copy full SHA for 3d83e0d - Browse repository at this point
Copy the full SHA 3d83e0dView commit details -
Fix -Wtype-equality-requires-operators warnings
GHC 9.4 adds `-Wtype-equality-requires-operators` to `-Wall`, which warns about certain uses of type equalities that are not forward-compatible with planned changes in GHC. See [this section](https://gitlab.haskell.org/ghc/ghc/-/wikis/migration/9.4?version_id=b60e52482a666d25638d59cd7e86851ddf971dc1#-is-now-a-type-operator) of the GHC 9.4 Migration Guide. These warnings are easily fixed by enabling the `TypeOperators` extension.
Configuration menu - View commit details
-
Copy full SHA for d54026d - Browse repository at this point
Copy the full SHA d54026dView commit details -
Remove use of NoMonoLocalBinds after GADTs pragma
This produces a `-Wgadt-mono-local-binds` warning with GHC 9.4, which has added the warning to `-Wall`. To avoid the warning, I have removed a use of `NoMonoLocalBinds` in `SAWScript.Crucible.JVM.BuiltinsJVM`. This required adding an explicit type signature to `failure` as a result.
Configuration menu - View commit details
-
Copy full SHA for d04c901 - Browse repository at this point
Copy the full SHA d04c901View commit details -
Configuration menu - View commit details
-
Copy full SHA for 3fe4fc8 - Browse repository at this point
Copy the full SHA 3fe4fc8View commit details -
Configuration menu - View commit details
-
Copy full SHA for eaaec00 - Browse repository at this point
Copy the full SHA eaaec00View commit details -
Configuration menu - View commit details
-
Copy full SHA for e19c9cb - Browse repository at this point
Copy the full SHA e19c9cbView commit details
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.