Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Enable dimension item/id/range dimension checks for DPC++ #751

Merged
merged 5 commits into from
Jul 25, 2023
Merged

Enable dimension item/id/range dimension checks for DPC++ #751

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
a852077
Enable dimension item/id/range dimension checks for DPC++
steffenlarsen Jul 18, 2023
d9cb0b2
Split id dimension checks to avoid kernel redefinitions
steffenlarsen Jul 19, 2023
7c08a0a
Merge branch 'SYCL-2020' into steffen/enable_dpcpp_dim_tests
steffenlarsen Jul 19, 2023
fdadb32
Add comment
steffenlarsen Jul 25, 2023
071103a
Merge branch 'SYCL-2020' into steffen/enable_dpcpp_dim_tests
steffenlarsen Jul 25, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 1 addition & 3 deletions tests/h_item/h_item_members.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -34,9 +34,7 @@ void run_test() {
range_index_space_id::check_members_test<sycl::h_item<Dim>, Dim>(type_name);
}

// FIXME: re-enable when sycl::h_item<>::dimensions is implemented
// Issue link https://github.com/intel/llvm/issues/9786
DISABLED_FOR_TEST_CASE(DPCPP, ComputeCpp)
DISABLED_FOR_TEST_CASE(ComputeCpp)
("sycl::h_item members", "[h_item]")({
run_test<1>();
run_test<2>();
Expand Down
4 changes: 1 addition & 3 deletions tests/id/id.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -284,9 +284,7 @@ TEMPLATE_TEST_CASE_SIG("id supports get() and operator[]", "[id]", ((int D), D),
}
}

// FIXME: re-enable when sycl::id<>::dimensions is implemented
// Issue link https://github.com/intel/llvm/issues/9786
DISABLED_FOR_TEMPLATE_TEST_CASE_SIG(DPCPP, hipSYCL)
DISABLED_FOR_TEMPLATE_TEST_CASE_SIG(hipSYCL)
("id provides static constexpr member 'dimensions'", "[id]", ((int D), D), 1, 2,
3)({
CHECK(std::is_same_v<decltype(sycl::id<D>::dimensions), const int>);
Expand Down
4 changes: 1 addition & 3 deletions tests/item/item_members.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -34,9 +34,7 @@ void run_test() {
range_index_space_id::check_members_test<sycl::item<Dim>, Dim>(type_name);
}

// FIXME: re-enable when sycl::item<>::dimensions is implemented
// Issue link https://github.com/intel/llvm/issues/9786
DISABLED_FOR_TEST_CASE(DPCPP, ComputeCpp)
DISABLED_FOR_TEST_CASE(ComputeCpp)
("sycl::item members", "[item]")({
run_test<1>();
run_test<2>();
Expand Down
7 changes: 2 additions & 5 deletions tests/nd_item/nd_item_members.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -34,13 +34,10 @@ void run_test() {
range_index_space_id::check_members_test<sycl::nd_item<Dim>, Dim>(type_name);
}

// FIXME: re-enable when sycl::nd_item<>::dimensions is implemented
// Issue link https://github.com/intel/llvm/issues/9786
DISABLED_FOR_TEST_CASE(DPCPP)
("sycl::nd_item members", "[nd_item]")({
TEST_CASE("sycl::nd_item members", "[nd_item]") {
run_test<1>();
run_test<2>();
run_test<3>();
});
}

} // namespace nd_item_members
4 changes: 1 addition & 3 deletions tests/nd_range/nd_range_members.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -34,9 +34,7 @@ void run_test() {
range_index_space_id::check_members_test<sycl::nd_range<Dim>, Dim>(type_name);
}

// FIXME: re-enable when sycl::nd_range<>::dimensions is implemented
// Issue link https://github.com/intel/llvm/issues/9786
DISABLED_FOR_TEST_CASE(DPCPP, ComputeCpp)
DISABLED_FOR_TEST_CASE(ComputeCpp)
("sycl::nd_range members", "[nd_range]")({
run_test<1>();
run_test<2>();
Expand Down
4 changes: 1 addition & 3 deletions tests/range/range_members.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -34,9 +34,7 @@ void run_test() {
range_index_space_id::check_members_test<sycl::range<Dim>, Dim>(type_name);
}

// FIXME: re-enable when sycl::range<>::dimensions is implemented
// Issue link https://github.com/intel/llvm/issues/9786
DISABLED_FOR_TEST_CASE(DPCPP, ComputeCpp)
DISABLED_FOR_TEST_CASE(ComputeCpp)
("sycl::range members", "[range]")({
run_test<1>();
run_test<2>();
Expand Down