Add properties parameter to all command-buffer commands#305
Add properties parameter to all command-buffer commands#305bashbaug merged 2 commits intoKhronosGroup:mainfrom
Conversation
8b3789d to
f5d6b98
Compare
|
I think it would be better to keep the current functions as they currently are, with no "properties" argument, and then add overloads with a "properties" argument. This has two main advantages:
We could even skip adding the overloads with the "properties" argument for now, at least for the command buffer APIs where no properties are currently defined. |
Updates to compile with header change KhronosGroup/OpenCL-Headers#260
f5d6b98 to
5a6632c
Compare
Avoiding breaking user code if we can avoid it makes sense to me. Updated and left out the overloaded properties variant. |
|
Merging as discussed in the September 3rd teleconference + email. The CI failures appear to be real, but are unrelated to these code changes. |
Updates to compile with header change KhronosGroup/OpenCL-Headers#260