-
Notifications
You must be signed in to change notification settings - Fork 2.3k
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
[async] Cache demote_activation #1889
Conversation
Codecov Report
@@ Coverage Diff @@
## master #1889 +/- ##
==========================================
- Coverage 43.80% 43.80% -0.01%
==========================================
Files 45 45
Lines 6168 6171 +3
Branches 1097 1097
==========================================
+ Hits 2702 2703 +1
- Misses 3297 3299 +2
Partials 169 169
Continue to review full report at Codecov.
|
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.
LGTM!
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.
Awesome! LGTM.
(Not sure if it's better to move IRBank to a separate file... This class is handling more and more stuff.)
Yes, that sounds like a helpful refactoring!
Co-authored-by: Yuanming Hu <yuanming-hu@users.noreply.github.com>
Related issue = #742
(Not sure if it's better to move
IRBank
to a separate file... This class is handling more and more stuff.)[Click here for the format server]