KhronosGroup / KhronosGroup/OpenCL-CTS

signed/unsigned mismatch in tested kernels.

Open
#1,698 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
C++
Stars
232
Forks
235
Avg merge
8d 7h
Merged PRs (30d)
18

Description

There are 557 instances of code like this:

```cpp
int tid = get_global_id(0);
```
But get_global_id returns size_t, so it's unsigned.

Some compilers (like Clspv) will warn on this.

The code should be updated. I don't have an opinion about whether tid should be changed to unsigned (or size_t), or whether to cast the value.

Contributor guide

No contributing guide indexed for this repository

Research direction

Search the tested kernels for instances such as `int tid = get_global_id(0)` and review how each value is used before choosing a consistent type or cast. Run the relevant OpenCL conformance tests and compiler checks to confirm the signed/unsigned warnings are addressed across the affected kernels.

Written by the indexing model from the issue text.

Assessment

Tech stack
cpp
Domain
testing-qa
Issue type
Refactor
Difficulty
5/5
Estimated time
Over a week
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.