-
Notifications
You must be signed in to change notification settings - Fork 406
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
Only copy data in UnorderedMap deep_copy #6812
Conversation
- Previous behavior was closer to create_mirror_view_and_copy - Now, no changes to meta data, only assert capacity matches and deep copy view data - add a create_mirror() function for UnorderedMaps
Fails on SYCL and HIP builds appear unrelated. I manually tested container unit tests on MI100 (caraway) and got passes for Serial and HIP tests. |
Retest this please |
1 similar comment
Retest this please |
I think the only fail is a CI issue, not with this PR. |
Only |
@dalg24 @tcclevenger does this merit an entry for the 4.4 changelog? Since it is a change of behavior, including deprecation, it seems notable to add? |
Deferring to Christian and Conrad. |
Yeah, this probably deserves an entry. I'll add one. |
Previously,
deep_copy(UnorderedMap,UnorderedMap)
acted much more similar tocreate_mirror_view_and_copy()
which allocated the dst view and then deep_copy the data. Now deep_copy requires the capacity of the src and dst UnorderedMaps are identical and deep copies all data from src to dst.Also changed
UnorderedMap::create_copy_view()
intoallocate_view()
anddeep_copy_view()
create_mirror(UnorderedMap)
specialization