-
Notifications
You must be signed in to change notification settings - Fork 113
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
[dpc++][ranges] adjacent_find, none_of, swap_ranges, copy_if... #202
Conversation
8e34720
to
c82358c
Compare
Seems not all algorithms are covered by tests. Do you plan to add tests for them to the patch? |
Yes, I'm going to add the missed tests. |
750f04c
to
7a596f5
Compare
21fdd24
to
ace3e9a
Compare
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 overall
44ea00e
to
ac05de3
Compare
a0f99e2
to
60f3f96
Compare
16e11d0
to
3d57394
Compare
6622184
to
da29051
Compare
…ue, unique_copy, and the tests
…g ranges with different sizes
…get_kernel_bundle" This reverts commit 450cff1.
11d5db9
to
3681f3e
Compare
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.
There are some issues on the infrastructure side but I don't have any questions to the patch itself. Looks good to me
* [dpc++][ranges] + none_of, adjacent_find * [dpc++][ranges] + copy_if, swap_ranges * [dpc++][ranges] + remove_copy, remove_copy_if * [dpc++][ranges] + unique, unique_copy * [dpc++][ranges] + clang format and fixing typos * [dpc++][ranges] + fix for none_of * [dpc++][ranges] + test for range-based none_of, adjacent_find * [dpc++][ranges] + reverse, reverse_copy, rotate_copy * [dpc++][ranges] + test for range-based swap_ranges * [dpc++][ranges] + views::host_all - a custom utility for data access in the host * [dpc++][ranges] + test for range-based copy_if * [dpc++][ranges] + using oneapi::dpl::__ranges::views::host_all * + clang format * [dpc++][ranges] + test for range-based remove_copy, remove_copy_if * [dpc++][ranges] + fixes for reverse, reverse_copy, rotate_copy * [dpc++][ranges] + test for range-based rotate_copy * [dpc++][pstl][tests] + EXPECT_EQ_RANGES utility * [dpc++][ranges] + fix for range-based rotate_copy * [dpc++][ranges] + range-based versions of reverse, reverse_copy, unique, unique_copy, and the tests * [dpc++][ranges][tests] + minor fixes * [dpc++][pstl][tests] + fix for test utils * [dpc++][ranges] + fix a typo; - extra empty lines * [dpc++][ranges] + using nano::subrange * [dpc++][ranges] + fix for range-based replace_copy, replace_copy_if * [dpc++][ranges] + tests for replace_copy, replace_copy_if * + clang format * [dpc++][pstl][tests] + minor change for copy_if_ranges_sycl.pass.cpp * [dpc++][ranges] + PR202 review comments have been addressed. * [dpc++][ranges] + minor changes in all_view_fn, all_view * [dpc++][ranges][tests] + constexpr * + clang format * [dpc++][ranges] + __get_min_size * [dpc++][ranges] + min size range as first for swap_ranges algorithm * [dpc++][ranges] extension of coverage for swap_ranges algo; processing ranges with different sizes * + clang format * [dpc++][ranges] + a fix for swap_ranges algorithm * [dpc++][ranges] + "test experiment" for swap_ranges_sycl.pass.cpp * [dpc++][ranges] + test coverage improvement (+ a predicate) * [dpc++][ranges] + revert "test experiment" for swap_ranges_sycl.pass.cpp * Revert "[dpc++][sycl] + passing sycl::bundle_state::object for sycl::get_kernel_bundle" This reverts commit 450cff1.
[dpc++][ranges] adjacent_find, none_of, swap_ranges, copy_if, remove_copy, remove_copy_if, unique, unique_copy, reverse, reverse_copy, rotate_copy