intel / intel/llvm

Enhance `nd_item` code coverage in CI validation.

Open
#12,563 0 comments 0 reactions 0 assignees View on GitHub
confirmed enhancement
Dominant language
LLVM
Stars
1.5k
Forks
854
Avg merge
3d 17h
Merged PRs (30d)
137

Description

During code review of https://github.com/intel/llvm/pull/12236, we found that `nd_item::get_nd_range()` is not covered by pre-commit tests. I think only Khronos SYCL CTS covers all `nd_item` methods, but unfortunately, intel/llvm CI doesn't run these tests.

I suggest we add a short "smoke" test to validate `nd_item::get_nd_range()` methods and other `nd_item` methods not covered by existing CI validation.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.