KhronosGroup / KhronosGroup/OpenCL-Headers

Breaking change rename of `cl_ndrange_kernel_command_properties_khr` to `cl_command_properties_khr`

Open
#266 3 comments 0 reactions 0 assignees View on GitHub
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

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.