-
Notifications
You must be signed in to change notification settings - Fork 98
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
RTS Refactoring #4204
RTS Refactoring #4204
Conversation
…ion0` in RTS unit tests, remove compiler warnings in RTS test
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, but didn't look that closely since mostly adjusting tests.
You seem to be merging the region0 root and continuation table which, to me, makes the code more obscure - what's the motive for this change. Does it make it less likey to forget the root?
Thanks for reducing cost of stable-region-migration test that often kills CI.
Co-authored-by: Claudio Russo <claudio@dfinity.org>
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.
"double-plus good"
Thank you for the review and feedback. |
memory_check
feature flag compilable again.region0
in RTS tests. (Btw: Theregion0
root has only been moved because it is not an array, not omitted. Previously, the declaredregion0
parameters were unused.)