-
Notifications
You must be signed in to change notification settings - Fork 139
[mlir] Summary-based activity analysis #2114
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
Conversation
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.
Looks reasonable overall, please give @wsmoses a chance to take a look
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.
Could a relevant test be added to make sure it doesn’t break?
b40b4d1
to
2e0a794
Compare
@pengmai I'm going to merge for now to avoid rebase issues, but comments above should be addressed subsequently |
d036184
to
63c2079
Compare
63c2079
to
6bf3dcd
Compare
6bf3dcd
to
c61bb1f
Compare
No description provided.