-
Notifications
You must be signed in to change notification settings - Fork 22
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
Umap multi-region support #77
Conversation
2. Replaced spindle log with one based on Umpire
Installing Store.hpp in correct location.
maintain umapsort in umap-apps.
#ifndef UMAP_RO_MODE | ||
page_address | ||
#endif | ||
) |
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.
The way I'm reading this, it will be void * followed by a right paren if it is read only mode. Is that what is intended?
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.
Correct. This is one way (perhaps not the best) to prevent compilation warnings of unused arguments when both type and argument are specified when the argument is not used. Specifying just the type maintains the signature, but no longer produces warnings when the arguments are ifdefed out in the implementation.
#ifndef UMAP_RO_MODE | ||
page_address | ||
#endif | ||
) |
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.
Same as previous comment about the void * (
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.
Couple of minor comments and questions. Otherwise, good to go!
This is a major update to umap including:
The API has also changed slightly and there is an associated umap-apps PR on the way as well.