KhronosGroup / KhronosGroup/OpenCL-CTS

Revisit CTS issue 939

Open
#970 1 comment 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Dominant language
C++
Stars
232
Forks
237
Avg merge
8d 7h
Merged PRs (30d)
18

Description

Revisit CTS issue https://github.com/KhronosGroup/OpenCL-CTS/issues/939 and the associated PR
https://github.com/KhronosGroup/OpenCL-CTS/pull/941

The summary for the original issue is
" The generator function for atomic_fence uses the current workgroup size to determine the number of non atomic variables per thread. However, when the workgroup size is modified (after the kernel is generated) we don't re-generate the kernel, leading to mismatches during verification."

The PR has been merged but we should revisit to determine if any additional improvements are needed.

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start by reading CTS issue 939 and merged PR 941, then trace the atomic_fence generator and its verification behavior described there. Determine whether the merged change leaves additional mismatches when the workgroup size changes; done means documenting or addressing any remaining improvements with appropriate CTS validation.

Written by the indexing model from the issue text.

Assessment

Tech stack
cpp
Domain
testing-qa
Issue type
Bug
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
25/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.