Tests for sycl::get_kernel_id and sycl::get_kernel_ids #218
Conversation
…ommon_by_reference.h
Following tests created: - tests for common-by-reference checks. - tests for sycl::kernel_id API
vasilytric
requested review from
bader,
illuhad,
keryell and
ProGTX
as code owners
| /** Check return type of sycl::get_kernel_id() | ||
| */ | ||
| run_verification<by_queue>(log, [&](sycl::queue &queue) { | ||
| using kernel_name = class k_id_ret_type; |
There was a problem hiding this comment.
Interesting, I have never thought to this syntax.
Addcomments for these functions: add_cts_test and add_independent_cts_tests
…urces files This could happen in case, when specified source *.cpp-files doesn't exist Previously generating process doesn't stopped in this case
These changes from the older code base
|
@vasilytric, CI says that computecpp and hipsycl doesn't support these features yet. |
Add kernel_bundle to them
@bader Oh, sorry. these files was updated in this commit: b4692ac |
|
You say that these translation units must not be linked together -- doesn't this mean that the |
Oh, this is a good question. The We also have another way:
|
|
@vasilytric, please, resolve merge conflicts. |
@bader Done |
|
@vasilytric, please, update pull requests depending on changes from this PR to get valid CI checks status. |
Some of these test required containing in separate *.exe-file:
get_kernel_ids_multiple_kernels,get_kernel_ids_single_kernelandget_kernel_ids_no_kernelsbecause these files required stricted list of available kernelsfor build this tests need call building process for each test separately:
make test_get_kernel_ids_multiple_kernels,make test_get_kernel_ids_single_kernelandmake test_get_kernel_ids_no_kernelsBut if use
makeall tests will be builded