From ae1884be423ef1a630569f84eae66cdaa1b478d5 Mon Sep 17 00:00:00 2001 From: Oleksandr Pavlyk Date: Sat, 9 Mar 2024 19:05:37 -0600 Subject: [PATCH 1/9] Improve coverage of dpctl_sycl_queue_interface Test submit for 1D range, 2D range, and 3D range, and also specify dependent events. --- .../tests/test_sycl_queue_submit.cpp | 34 +++++++++++++++---- 1 file changed, 28 insertions(+), 6 deletions(-) diff --git a/libsyclinterface/tests/test_sycl_queue_submit.cpp b/libsyclinterface/tests/test_sycl_queue_submit.cpp index cc9bc836ce..8330d6da1e 100644 --- a/libsyclinterface/tests/test_sycl_queue_submit.cpp +++ b/libsyclinterface/tests/test_sycl_queue_submit.cpp @@ -56,7 +56,9 @@ void submit_kernel(DPCTLSyclQueueRef QRef, { T scalarVal = 3; constexpr size_t NARGS = 4; - constexpr size_t RANGE_NDIMS = 1; + constexpr size_t RANGE_NDIMS_1 = 1; + constexpr size_t RANGE_NDIMS_2 = 2; + constexpr size_t RANGE_NDIMS_3 = 3; ASSERT_TRUE(DPCTLKernelBundle_HasKernel(KBRef, kernelName.c_str())); auto kernel = DPCTLKernelBundle_GetKernel(KBRef, kernelName.c_str()); @@ -75,13 +77,33 @@ void submit_kernel(DPCTLSyclQueueRef QRef, (void *)&scalarVal}; DPCTLKernelArgType addKernelArgTypes[] = {DPCTL_VOID_PTR, DPCTL_VOID_PTR, DPCTL_VOID_PTR, kernelArgTy}; - auto ERef = DPCTLQueue_SubmitRange(kernel, QRef, args, addKernelArgTypes, - NARGS, Range, RANGE_NDIMS, nullptr, 0); - ASSERT_TRUE(ERef != nullptr); - DPCTLQueue_Wait(QRef); + auto E1Ref = + DPCTLQueue_SubmitRange(kernel, QRef, args, addKernelArgTypes, NARGS, + Range, RANGE_NDIMS_1, nullptr, 0); + ASSERT_TRUE(E1Ref != nullptr); + + // Create kernel args for vector_add + size_t Range2D[] = {SIZE, 1}; + DPCTLSyclEventRef DepEvs[] = {E1Ref}; + auto E2Ref = + DPCTLQueue_SubmitRange(kernel, QRef, args, addKernelArgTypes, NARGS, + Range2D, RANGE_NDIMS_2, DepEvs, 1); + ASSERT_TRUE(E2Ref != nullptr); + + // Create kernel args for vector_add + size_t Range3D[] = {SIZE, 1, 1}; + DPCTLSyclEventRef DepEvs2[] = {E1Ref, E2Ref}; + auto E3Ref = + DPCTLQueue_SubmitRange(kernel, QRef, args, addKernelArgTypes, NARGS, + Range3D, RANGE_NDIMS_3, DepEvs2, 2); + ASSERT_TRUE(E3Ref != nullptr); + + DPCTLEvent_Wait(E3Ref); // clean ups - DPCTLEvent_Delete(ERef); + DPCTLEvent_Delete(E1Ref); + DPCTLEvent_Delete(E2Ref); + DPCTLEvent_Delete(E3Ref); DPCTLKernel_Delete(kernel); DPCTLfree_with_queue((DPCTLSyclUSMRef)a, QRef); DPCTLfree_with_queue((DPCTLSyclUSMRef)b, QRef); From 78d55c87405a55fa836231c31c9acce4e1610372 Mon Sep 17 00:00:00 2001 From: Oleksandr Pavlyk Date: Sat, 9 Mar 2024 19:06:44 -0600 Subject: [PATCH 2/9] host is no longer a valid string selector --- libsyclinterface/tests/test_sycl_device_aspects.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libsyclinterface/tests/test_sycl_device_aspects.cpp b/libsyclinterface/tests/test_sycl_device_aspects.cpp index 61f0eaba8b..093c6730df 100644 --- a/libsyclinterface/tests/test_sycl_device_aspects.cpp +++ b/libsyclinterface/tests/test_sycl_device_aspects.cpp @@ -97,7 +97,7 @@ auto build_gtest_values(const std::array, N> ¶ms) auto build_params() { constexpr auto param_1 = get_param_list( - "opencl:gpu", "opencl:cpu", "level_zero:gpu", "host"); + "opencl:gpu", "opencl:cpu", "level_zero:gpu"); constexpr auto param_2 = get_param_list>( From 24574220e092510112b7de2a93390033f5dad08e Mon Sep 17 00:00:00 2001 From: Oleksandr Pavlyk Date: Mon, 11 Mar 2024 03:58:29 -0500 Subject: [PATCH 3/9] Add test for DPCTLQueue_Create for some nullptr args --- .../tests/test_sycl_queue_interface.cpp | 28 +++++++++++++++++++ 1 file changed, 28 insertions(+) diff --git a/libsyclinterface/tests/test_sycl_queue_interface.cpp b/libsyclinterface/tests/test_sycl_queue_interface.cpp index cddc007f4d..1c7fe55561 100644 --- a/libsyclinterface/tests/test_sycl_queue_interface.cpp +++ b/libsyclinterface/tests/test_sycl_queue_interface.cpp @@ -90,6 +90,34 @@ struct TestDPCTLQueueMemberFunctions } /* End of anonymous namespace */ +TEST(TestDPCTLSyclQueueInterface, CheckCreate) +{ + /* We are testing that we do not crash even when input is NULL. */ + DPCTLSyclQueueRef QRef = nullptr; + + EXPECT_NO_FATAL_FAILURE( + QRef = DPCTLQueue_Create(nullptr, nullptr, nullptr, 0)); + ASSERT_TRUE(QRef == nullptr); +} + +TEST(TestDPCTLSyclQueueInterface, CheckCreate2) +{ + /* We are testing that we do not crash even when input is NULL. */ + DPCTLSyclQueueRef QRef = nullptr; + DPCTLSyclDeviceSelectorRef DSRef = nullptr; + DPCTLSyclDeviceRef DRef = nullptr; + + EXPECT_NO_FATAL_FAILURE(DSRef = DPCTLDefaultSelector_Create()); + EXPECT_NO_FATAL_FAILURE(DRef = DPCTLDevice_CreateFromSelector(DSRef)); + EXPECT_NO_FATAL_FAILURE(DPCTLDeviceSelector_Delete(DSRef)); + + EXPECT_NO_FATAL_FAILURE(QRef = + DPCTLQueue_Create(nullptr, DRef, nullptr, 0)); + ASSERT_TRUE(QRef == nullptr); + + EXPECT_NO_FATAL_FAILURE(DPCTLDevice_Delete(DRef)); +} + TEST(TestDPCTLSyclQueueInterface, CheckCreateForDevice) { /* We are testing that we do not crash even when input is NULL. */ From 0a12c3156b7ba1070b88f8c62d33de86eb3a456a Mon Sep 17 00:00:00 2001 From: Oleksandr Pavlyk Date: Mon, 11 Mar 2024 03:59:17 -0500 Subject: [PATCH 4/9] Only run test submit if device has aspect fp64 --- libsyclinterface/tests/test_sycl_queue_submit.cpp | 12 +++++++----- .../test_sycl_queue_submit_local_accessor_arg.cpp | 12 +++++++----- 2 files changed, 14 insertions(+), 10 deletions(-) diff --git a/libsyclinterface/tests/test_sycl_queue_submit.cpp b/libsyclinterface/tests/test_sycl_queue_submit.cpp index 8330d6da1e..d89ec3d3ce 100644 --- a/libsyclinterface/tests/test_sycl_queue_submit.cpp +++ b/libsyclinterface/tests/test_sycl_queue_submit.cpp @@ -256,13 +256,13 @@ struct TestQueueSubmitFP64 : public ::testing::Test std::ifstream spirvFile; size_t spirvFileSize_; std::vector spirvBuffer_; + DPCTLSyclDeviceRef DRef = nullptr; DPCTLSyclQueueRef QRef = nullptr; DPCTLSyclKernelBundleRef KBRef = nullptr; TestQueueSubmitFP64() { DPCTLSyclDeviceSelectorRef DSRef = nullptr; - DPCTLSyclDeviceRef DRef = nullptr; spirvFile.open("./oneD_range_kernel_fp64.spv", std::ios::binary | std::ios::ate); @@ -279,13 +279,13 @@ struct TestQueueSubmitFP64 : public ::testing::Test KBRef = DPCTLKernelBundle_CreateFromSpirv( CRef, DRef, spirvBuffer_.data(), spirvFileSize_, nullptr); - DPCTLDevice_Delete(DRef); DPCTLDeviceSelector_Delete(DSRef); } ~TestQueueSubmitFP64() { spirvFile.close(); + DPCTLDevice_Delete(DRef); DPCTLQueue_Delete(QRef); DPCTLKernelBundle_Delete(KBRef); } @@ -356,9 +356,11 @@ TEST_F(TestQueueSubmit, CheckForFloat) TEST_F(TestQueueSubmitFP64, CheckForDouble) { - submit_kernel(QRef, KBRef, spirvBuffer_, spirvFileSize_, - DPCTLKernelArgType::DPCTL_FLOAT64_T, - "_ZTS11RangeKernelIdE"); + if (DPCTLDevice_HasAspect(DRef, DPCTLSyclAspectType::fp64)) { + submit_kernel(QRef, KBRef, spirvBuffer_, spirvFileSize_, + DPCTLKernelArgType::DPCTL_FLOAT64_T, + "_ZTS11RangeKernelIdE"); + } } TEST_F(TestQueueSubmit, CheckForUnsupportedArgTy) diff --git a/libsyclinterface/tests/test_sycl_queue_submit_local_accessor_arg.cpp b/libsyclinterface/tests/test_sycl_queue_submit_local_accessor_arg.cpp index b14bef9e7b..0d39c4b281 100644 --- a/libsyclinterface/tests/test_sycl_queue_submit_local_accessor_arg.cpp +++ b/libsyclinterface/tests/test_sycl_queue_submit_local_accessor_arg.cpp @@ -239,13 +239,13 @@ struct TestQueueSubmitWithLocalAccessorFP64 : public ::testing::Test std::ifstream spirvFile; size_t spirvFileSize_; std::vector spirvBuffer_; + DPCTLSyclDeviceRef DRef = nullptr; DPCTLSyclQueueRef QRef = nullptr; DPCTLSyclKernelBundleRef KBRef = nullptr; TestQueueSubmitWithLocalAccessorFP64() { DPCTLSyclDeviceSelectorRef DSRef = nullptr; - DPCTLSyclDeviceRef DRef = nullptr; spirvFile.open("./local_accessor_kernel_fp64.spv", std::ios::binary | std::ios::ate); @@ -262,13 +262,13 @@ struct TestQueueSubmitWithLocalAccessorFP64 : public ::testing::Test KBRef = DPCTLKernelBundle_CreateFromSpirv( CRef, DRef, spirvBuffer_.data(), spirvFileSize_, nullptr); - DPCTLDevice_Delete(DRef); DPCTLDeviceSelector_Delete(DSRef); } ~TestQueueSubmitWithLocalAccessorFP64() { spirvFile.close(); + DPCTLDevice_Delete(DRef); DPCTLQueue_Delete(QRef); DPCTLKernelBundle_Delete(KBRef); } @@ -339,9 +339,11 @@ TEST_F(TestQueueSubmitWithLocalAccessor, CheckForFloat) TEST_F(TestQueueSubmitWithLocalAccessorFP64, CheckForDouble) { - submit_kernel(QRef, KBRef, spirvBuffer_, spirvFileSize_, - DPCTLKernelArgType::DPCTL_FLOAT64_T, - "_ZTS14SyclKernel_SLMIdE"); + if (DPCTLDevice_HasAspect(DRef, DPCTLSyclAspectType::fp64)) { + submit_kernel(QRef, KBRef, spirvBuffer_, spirvFileSize_, + DPCTLKernelArgType::DPCTL_FLOAT64_T, + "_ZTS14SyclKernel_SLMIdE"); + } } TEST_F(TestQueueSubmitWithLocalAccessor, CheckForUnsupportedArgTy) From 2c85fa77ff17c717c987604e6f9cc53734e7bb30 Mon Sep 17 00:00:00 2001 From: Oleksandr Pavlyk Date: Mon, 11 Mar 2024 04:02:49 -0500 Subject: [PATCH 5/9] Test Local accessor for ndim=1, 2, 3 --- ...t_sycl_queue_submit_local_accessor_arg.cpp | 37 +++++++++++++++---- 1 file changed, 29 insertions(+), 8 deletions(-) diff --git a/libsyclinterface/tests/test_sycl_queue_submit_local_accessor_arg.cpp b/libsyclinterface/tests/test_sycl_queue_submit_local_accessor_arg.cpp index 0d39c4b281..8fa8e828d0 100644 --- a/libsyclinterface/tests/test_sycl_queue_submit_local_accessor_arg.cpp +++ b/libsyclinterface/tests/test_sycl_queue_submit_local_accessor_arg.cpp @@ -75,20 +75,39 @@ void submit_kernel(DPCTLSyclQueueRef QRef, a_ptr[i] = 0; } - auto la = MDLocalAccessor{1, kernelArgTy, SIZE / 10, 1, 1}; + auto la1 = MDLocalAccessor{1, kernelArgTy, SIZE / 10, 1, 1}; // Create kernel args for vector_add size_t gRange[] = {SIZE}; size_t lRange[] = {SIZE / 10}; - void *args[NARGS] = {unwrap(a), (void *)&la}; + void *args_1d[NARGS] = {unwrap(a), (void *)&la1}; DPCTLKernelArgType addKernelArgTypes[] = {DPCTL_VOID_PTR, DPCTL_LOCAL_ACCESSOR}; - auto ERef = - DPCTLQueue_SubmitNDRange(kernel, QRef, args, addKernelArgTypes, NARGS, - gRange, lRange, RANGE_NDIMS, nullptr, 0); - ASSERT_TRUE(ERef != nullptr); - DPCTLQueue_Wait(QRef); + DPCTLSyclEventRef E1Ref = DPCTLQueue_SubmitNDRange( + kernel, QRef, args_1d, addKernelArgTypes, NARGS, gRange, lRange, + RANGE_NDIMS, nullptr, 0); + ASSERT_TRUE(E1Ref != nullptr); + + DPCTLSyclEventRef DepEv1[] = {E1Ref}; + auto la2 = MDLocalAccessor{2, kernelArgTy, SIZE / 10, 1, 1}; + void *args_2d[NARGS] = {unwrap(a), (void *)&la2}; + + DPCTLSyclEventRef E2Ref = + DPCTLQueue_SubmitNDRange(kernel, QRef, args_2d, addKernelArgTypes, + NARGS, gRange, lRange, RANGE_NDIMS, DepEv1, 1); + ASSERT_TRUE(E2Ref != nullptr); + + DPCTLSyclEventRef DepEv2[] = {E1Ref, E2Ref}; + auto la3 = MDLocalAccessor{3, kernelArgTy, SIZE / 10, 1, 1}; + void *args_3d[NARGS] = {unwrap(a), (void *)&la3}; + + DPCTLSyclEventRef E3Ref = + DPCTLQueue_SubmitNDRange(kernel, QRef, args_3d, addKernelArgTypes, + NARGS, gRange, lRange, RANGE_NDIMS, DepEv2, 2); + ASSERT_TRUE(E3Ref != nullptr); + + DPCTLEvent_Wait(E3Ref); if (kernelArgTy != DPCTL_FLOAT32_T && kernelArgTy != DPCTL_FLOAT64_T) ASSERT_TRUE(a_ptr[0] == 20); @@ -96,7 +115,9 @@ void submit_kernel(DPCTLSyclQueueRef QRef, ASSERT_TRUE(a_ptr[0] == 20.0); // clean ups - DPCTLEvent_Delete(ERef); + DPCTLEvent_Delete(E1Ref); + DPCTLEvent_Delete(E2Ref); + DPCTLEvent_Delete(E3Ref); DPCTLKernel_Delete(kernel); DPCTLfree_with_queue((DPCTLSyclUSMRef)a, QRef); } From 05677e6ffdbc1e8f9dc72150102e7de684c6545a Mon Sep 17 00:00:00 2001 From: Oleksandr Pavlyk Date: Mon, 11 Mar 2024 04:05:03 -0500 Subject: [PATCH 6/9] Renamed target llvm-cov to llvm-cov-report, added target llvm-cov-show The llvm-cov-report outputs the coverage summary, while llvm-cov-show displays per-line coverage statistics to identify missed lines. --- libsyclinterface/tests/CMakeLists.txt | 34 +++++++++++++++++++++++---- 1 file changed, 30 insertions(+), 4 deletions(-) diff --git a/libsyclinterface/tests/CMakeLists.txt b/libsyclinterface/tests/CMakeLists.txt index 19eed6c06f..4a991340e7 100644 --- a/libsyclinterface/tests/CMakeLists.txt +++ b/libsyclinterface/tests/CMakeLists.txt @@ -89,8 +89,35 @@ if(DPCTL_GENERATE_COVERAGE) ${CMAKE_DL_LIBS} ) set(object_arg "-object;") - add_custom_target(llvm-cov + add_custom_target(run-c-api-tests COMMAND ${CMAKE_COMMAND} -E env DPCTL_VERBOSITY=warning ${CMAKE_CURRENT_BINARY_DIR}/dpctl_c_api_tests + WORKING_DIRECTORY ${CMAKE_CURRENT_BINARY_DIR} + COMMAND_EXPAND_LISTS + DEPENDS dpctl_c_api_tests + ) + add_custom_target(llvm-cov-show + COMMAND ${LLVMProfdata_EXE} + merge + -sparse default.profraw + -o + dpctl.profdata + COMMAND ${LLVMCov_EXE} + export + -format=lcov + -ignore-filename-regex=/tmp/icpx* + -instr-profile=dpctl.profdata + "${object_arg}$,;${object_arg}>" + > dpctl.lcov + COMMAND ${LLVMCov_EXE} + show + -instr-profile=dpctl.profdata + "${object_arg}$,;${object_arg}>" + WORKING_DIRECTORY ${CMAKE_CURRENT_BINARY_DIR} + COMMAND_EXPAND_LISTS + DEPENDS run-c-api-tests + ) + + add_custom_target(llvm-cov-report COMMAND ${LLVMProfdata_EXE} merge -sparse default.profraw @@ -109,11 +136,10 @@ if(DPCTL_GENERATE_COVERAGE) "${object_arg}$,;${object_arg}>" WORKING_DIRECTORY ${CMAKE_CURRENT_BINARY_DIR} COMMAND_EXPAND_LISTS - DEPENDS dpctl_c_api_tests + DEPENDS run-c-api-tests ) add_custom_target(lcov-genhtml - COMMAND ${CMAKE_COMMAND} -E env DPCTL_VERBOSITY=warning ${CMAKE_CURRENT_BINARY_DIR}/dpctl_c_api_tests COMMAND ${LLVMProfdata_EXE} merge -sparse default.profraw @@ -132,7 +158,7 @@ if(DPCTL_GENERATE_COVERAGE) ${COVERAGE_OUTPUT_DIR}/dpctl-c-api-coverage WORKING_DIRECTORY ${CMAKE_CURRENT_BINARY_DIR} COMMAND_EXPAND_LISTS - DEPENDS dpctl_c_api_tests + DEPENDS run-c-api-tests ) else() target_link_libraries(dpctl_c_api_tests From 30661fdaa54f5eac2df68920cfa82257dcefbaa9 Mon Sep 17 00:00:00 2001 From: Oleksandr Pavlyk Date: Mon, 11 Mar 2024 04:09:53 -0500 Subject: [PATCH 7/9] Update to dbg_build Let LLVM_TOOLS_HOME and PATH to make llvm-cov, and llvm-profdata discoverable by cmake. --- libsyclinterface/dbg_build.sh | 15 +++++++++++++-- 1 file changed, 13 insertions(+), 2 deletions(-) diff --git a/libsyclinterface/dbg_build.sh b/libsyclinterface/dbg_build.sh index f036796a2e..cba4e71d71 100755 --- a/libsyclinterface/dbg_build.sh +++ b/libsyclinterface/dbg_build.sh @@ -7,6 +7,11 @@ pushd build || exit 1 INSTALL_PREFIX=$(pwd)/../install rm -rf ${INSTALL_PREFIX} +# With DPC++ 2024.0 adn newer set these to ensure that +# cmake can find llvm-cov and other utilities +LLVM_TOOLS_HOME=${CMPLR_ROOT}/bin/compiler +PATH=$PATH:${CMPLR_ROOT}/bin/compiler + cmake \ -DCMAKE_BUILD_TYPE=Debug \ -DCMAKE_C_COMPILER=icx \ @@ -16,13 +21,19 @@ cmake \ -DCMAKE_PREFIX_PATH=${INSTALL_PREFIX} \ -DDPCTL_ENABLE_L0_PROGRAM_CREATION=ON \ -DDPCTL_BUILD_CAPI_TESTS=ON \ + -DDPCTL_GENERATE_COVERAGE=OFF \ .. -make V=1 -n -j 4 && make check && make install +# build +make V=1 -n -j 4 +# run ctest +make check +# install +make install # Turn on to generate coverage report html files reconfigure with # -DDPCTL_GENERATE_COVERAGE=ON and then -# make lcov-genhtml +# make llvm-cov-report # For more verbose tests use: # cd tests From 278fec209421d21365bb26e11ab8498316e90899 Mon Sep 17 00:00:00 2001 From: Oleksandr Pavlyk Date: Mon, 11 Mar 2024 04:51:36 -0500 Subject: [PATCH 8/9] Change script to use llvm-cov-report instead of llvm-cov --- scripts/gen_coverage.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/scripts/gen_coverage.py b/scripts/gen_coverage.py index fece9228fd..e81f7b301b 100644 --- a/scripts/gen_coverage.py +++ b/scripts/gen_coverage.py @@ -82,7 +82,7 @@ def run( .strip("\n") ) subprocess.check_call( - ["cmake", "--build", ".", "--target", "llvm-cov"], + ["cmake", "--build", ".", "--target", "llvm-cov-report"], cwd=cmake_build_dir, ) env["LLVM_PROFILE_FILE"] = "dpctl_pytest.profraw" From bbb2d5401cf8dc452fcad76880cf3ed26d7d54b0 Mon Sep 17 00:00:00 2001 From: Oleksandr Pavlyk Date: Mon, 11 Mar 2024 08:35:31 -0500 Subject: [PATCH 9/9] Moved typedef for MDLocalAccessor to dpctl_sycl_queue_interface.hpp --- .../include/dpctl_sycl_queue_interface.h | 12 ++++++++++++ .../source/dpctl_sycl_queue_interface.cpp | 9 --------- .../test_sycl_queue_submit_local_accessor_arg.cpp | 9 --------- 3 files changed, 12 insertions(+), 18 deletions(-) diff --git a/libsyclinterface/include/dpctl_sycl_queue_interface.h b/libsyclinterface/include/dpctl_sycl_queue_interface.h index 18d55808ea..1763e1d2d5 100644 --- a/libsyclinterface/include/dpctl_sycl_queue_interface.h +++ b/libsyclinterface/include/dpctl_sycl_queue_interface.h @@ -171,6 +171,18 @@ DPCTL_API __dpctl_give DPCTLSyclDeviceRef DPCTLQueue_GetDevice(__dpctl_keep const DPCTLSyclQueueRef QRef); +/*! @brief Structure to be used to specify dimensionality and type of + * local_accessor kernel type argument. + */ +typedef struct MDLocalAccessorTy +{ + size_t ndim; + DPCTLKernelArgType dpctl_type_id; + size_t dim0; + size_t dim1; + size_t dim2; +} MDLocalAccessor; + /*! * @brief Submits the kernel to the specified queue with the provided range * argument. diff --git a/libsyclinterface/source/dpctl_sycl_queue_interface.cpp b/libsyclinterface/source/dpctl_sycl_queue_interface.cpp index 04ab441332..dce5a06a99 100644 --- a/libsyclinterface/source/dpctl_sycl_queue_interface.cpp +++ b/libsyclinterface/source/dpctl_sycl_queue_interface.cpp @@ -121,15 +121,6 @@ typedef struct complex uint64_t imag; } complexNumber; -typedef struct MDLocalAccessorTy -{ - size_t ndim; - DPCTLKernelArgType dpctl_type_id; - size_t dim0; - size_t dim1; - size_t dim2; -} MDLocalAccessor; - void set_dependent_events(handler &cgh, __dpctl_keep const DPCTLSyclEventRef *DepEvents, size_t NDepEvents) diff --git a/libsyclinterface/tests/test_sycl_queue_submit_local_accessor_arg.cpp b/libsyclinterface/tests/test_sycl_queue_submit_local_accessor_arg.cpp index 8fa8e828d0..7f28fc0041 100644 --- a/libsyclinterface/tests/test_sycl_queue_submit_local_accessor_arg.cpp +++ b/libsyclinterface/tests/test_sycl_queue_submit_local_accessor_arg.cpp @@ -44,15 +44,6 @@ constexpr size_t SIZE = 100; using namespace dpctl::syclinterface; -typedef struct MDLocalAccessorTy -{ - size_t ndim; - DPCTLKernelArgType dpctl_type_id; - size_t dim0; - size_t dim1; - size_t dim2; -} MDLocalAccessor; - template void submit_kernel(DPCTLSyclQueueRef QRef, DPCTLSyclKernelBundleRef KBRef,