-
Notifications
You must be signed in to change notification settings - Fork 27
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
Adds support for RAJA and Umpire to Windows build #987
Conversation
And incorporates them into the axom (host-config generator) port.
…orts OpenMP is enabled by default for axom's port.
Using [email protected] due to a bug with zero-sized allocations in v2022.03.
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.
Thanks very much, @kennyweiss .
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 good to me, but I don't work with vcpkg stuff. So my approval is minimally useful :-)
Passed the Windows TPL build CI: https://github.com/LLNL/axom/actions/runs/3842574364 |
Summary
vcpkg
ports forRAJA
,Umpire
andcamp
fmt
version due to an incompatibility between the vendored copies offmt
inaxom
andUmpire
. Specifically, the strerror overloads in[email protected]/format-inl.h
differed only in return type. These were removed in[email protected]
OpenMP
feature to the Windows build