-
Notifications
You must be signed in to change notification settings - Fork 36
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
Move axl mpi to axl #501
Move axl mpi to axl #501
Conversation
This is a refactoring step in preparation for being able to utilize MPI when flushing shared files.
This one is hard to test until we start to merge things, but we'll want to remember to move this line: Line 274 in d153d58
|
Nice catch! I mistakenly left that in. I also need to modify the dist script to include MPI=On when building AXL (if needed). Let me change that now |
@adammoody, for dist, do I need to create both a noMPI library in addition to one with MPI enabled (similar to what is being done for other libraries like redset and others? |
Yeah, I was just looking at that. I think we will need to create a noMPI section for AXL, since we use AXL in the serial executables, which should not have MPI. Maybe just copy the kvtree pattern again. This should be easier to test when we merge the AXL PR. |
This relates to #489
The relates to AXL PR: ECP-VeloC/AXL#129