oneapi-src / oneapi-src/unified-runtime
Add RAII handle wrappers to aid UR users
Nobody has claimed this yet.
- Dominant language
- C++
- Stars
- 57
- Forks
- 120
- Avg merge
- 1d 14h
- Merged PRs (30d)
- 1
Description
There are at least 3 places where RAII wrappers for UR handles have been either implemented or requested recently:
- https://github.com/oneapi-src/unified-runtime/blob/main/test/conformance/testing/include/uur/raii.h
- https://github.com/oneapi-src/unified-runtime/pull/1286/files#diff-d0ae1ee0edc109c505db620df503c365a6daf95b5f4ef9f12ba06a7d16e0b3d8
- https://github.com/intel/llvm/pull/12589#discussion_r1475997415
Given this is a common pattern in C++ code, and because it would be easy to generate, we should provide RAII wrappers as a public header that can be reused rather than implemented in multiple places.
Contributor guide
No contributing guide indexed for this repository
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start by reading the existing RAII wrappers in test/conformance/testing/include/uur/raii.h and compare the implementations and requests linked in pull requests 1286 and 12589. Define a reusable public header that covers the UR handle patterns described in those references and avoids separate implementations in each user. Done means UR users can include the public wrappers instead of recreating them.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- cpp
- Domain
- api
- Issue type
- Feature
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100