-
Notifications
You must be signed in to change notification settings - Fork 88
Modular explicits #2456
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
Open
samsa1
wants to merge
25
commits into
ocaml-flambda:main
Choose a base branch
from
samsa1:modular-explicit
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Modular explicits #2456
Changes from all commits
Commits
Show all changes
25 commits
Select commit
Hold shift + click to select a range
ca113a8
Initial port without typecore
samsa1 5fa1a40
Improved did typing of unknown arg
samsa1 b91b67b
Finished porting minimal version of modular explicits
samsa1 05c5442
Corrected errors in printer/printast_with_mappings.ml
samsa1 ac77fa6
Fixed build process, tests are failling due to printing errors
samsa1 14d1be4
Updated tests to match current implementation
samsa1 83ef0be
modified chamelon to be able to compile
samsa1 dad9cf6
Revert "modified chamelon to be able to compile"
samsa1 27ca01c
Fixed chamelon to compile easily
samsa1 18b8497
Added example of non-commutativity of typing
samsa1 d8e0148
Solved some bugs and detected a bug related to layout
samsa1 f2ff135
Updated syntax to be compatible with first class modules and resolved…
samsa1 a1e5299
Cleaned code of update_level
samsa1 a7a08d6
Corrected subtype_rec with Tfunctors
samsa1 5544a61
Cleaned up a bit of code and corrected a few bugs
samsa1 6c942f0
Backported multiple bug fixes
e9529fb
Fixes post rebase
ba7b05a
Corrected layout error
samsa1 b985076
First pass on reviews
samsa1 b71c160
Second pass on commets, mostly about tests
samsa1 a528f0a
Second pass on tests
samsa1 33ceaee
Moved things around in ident.ml and refactored a bit in path.ml
samsa1 504cb2b
Corrected tests in syntactic-arity/warnings.ml
samsa1 e92698d
Refactoring in typecore and improvement in error messages
samsa1 857be89
Major rework to handle modes, however this broke printing with rectypes
samsa1 File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think it should be pretty easy to support modes on these, and you'll need to so that you can give a most general type to functions with first-class module parameters. Here you just need to copy the mode stuff from the code just above.