KhronosGroup / KhronosGroup/OpenCL-Headers
Breaking change rename of `cl_ndrange_kernel_command_properties_khr` to `cl_command_properties_khr`
- Dominant language
- C
- Stars
- 766
- Forks
- 273
- Avg merge
- 2d 2h
- Merged PRs (30d)
- 3
Description
The recent [headers update](https://github.com/KhronosGroup/OpenCL-Headers/commit/862f06c64c222b683f22a872120fc066040dd2c8) includes a rename of `cl_ndrange_kernel_command_properties_khr` to `cl_command_properties_khr`. This broke a bunch of targets when importing this repo into Google's repos, including TensorFlow.
Please add a using alias of `cl_ndrange_kernel_command_properties_khr` to `cl_command_properties_khr`? You can aslo add a `[[deprecated]]` attribute to the using alias so that a warning shows up, allowing clients to update their code.
Contributor guide
No contributing guide indexed for this repository
Research direction
Start with the OpenCL header declarations affected by the headers update in commit 862f06c64c222b683f22a872120fc066040dd2c8. Add compatibility for the old cl_ndrange_kernel_command_properties_khr name, optionally marking it deprecated, and confirm that clients using the old name can still import the headers while receiving the intended warning.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- c
- Domain
- api
- Issue type
- Bug
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 45/100