Skip to content
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

Shape analysis #58

Draft
wants to merge 133 commits into
base: master
Choose a base branch
from
Draft

Shape analysis #58

wants to merge 133 commits into from

Conversation

lucaspena
Copy link
Contributor

OOPSLA (new kt-collapse rule, shape analysis rules, abstracting nil, etc.)

@xc93
Copy link
Collaborator

xc93 commented Apr 16, 2020

What's the plan for this PR? It looks too big to be merged given its current status.

@nishantjr
Copy link
Member

Yes, we're putting it here so that the CI runs the tests. We're going to break this apart into multiple PRs later (the subst PR is one of them)

This fixes issues in the various drivers. This logically makes more sense
since it is the strategy cell that controls the execution.
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