-
Notifications
You must be signed in to change notification settings - Fork 116
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
WIP: Various tweaks to the Rewriters #548
Merged
Merged
Changes from 20 commits
Commits
Show all changes
24 commits
Select commit
Hold shift + click to select a range
5e883dd
forward metadata in Walk
shashi 657bd9b
create special slot ~MATCH which is only defined on the RHS and holds…
shashi 80bf968
Merge branch 's/inspect' into s/rewriter-tweaks
shashi 9a6e04e
allow early stop in Chain
shashi 5de0612
add inspect for non-symbolic objects so user code does not have to check
shashi 3b53344
prevent canonicalization and hence destruction of terms with metadata
shashi 1a89dd5
fix check in +
shashi 1517c40
fix typo
shashi 9ce7c76
forward metadata in Walk
shashi 05c87d4
create special slot ~MATCH which is only defined on the RHS and holds…
shashi 4885f7b
allow early stop in Chain
shashi 8c3b506
prevent canonicalization and hence destruction of terms with metadata
shashi 1712ba9
fix check in +
shashi a1b8cf9
fix typo
shashi 93545fb
remove redundant and wrpng nometa check
shashi 5fa8545
messy rebase
Vaibhavdixit02 ec0383d
Messy rebase
Vaibhavdixit02 3680e8a
Merge branch 'master' into s/rewriter-tweaks
Vaibhavdixit02 a9867cc
Merge remote-tracking branch 'origin/s/rewriter-tweaks' into s/rewrit…
shashi 0fd9fbe
left out line
shashi 993d1fa
comment out repeated istree check
Vaibhavdixit02 f387fe6
Bring back istree check in walking
Vaibhavdixit02 6f76a28
Add test and fix branch in walk
Vaibhavdixit02 1457894
more test
Vaibhavdixit02 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 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 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.
This looks incorrect right?
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.
yes